Melhora escrita em log

- Corrige locais que escreviam stacktrace no log. - Novas constantes criadas.

Melhora escrita em log
- Corrige locais que escreviam stacktrace no log. - Novas constantes criadas.
andre.guimaraes
1 parent dec392b2
Showing 6 changed files with 16 additions and 13 deletions Show diff stats
pom.xml
src/main/java/br/gov/ans/integracao/sei/rest/ContatoResource.java
src/main/java/br/gov/ans/integracao/sei/rest/DocumentoResource.java
src/main/java/br/gov/ans/integracao/sei/rest/MessagesKeys.java
src/main/java/br/gov/ans/integracao/sei/rest/ProcessoResource.java
src/main/java/br/gov/ans/integracao/sei/utils/MessagesKeys.java
@@ -4,7 +4,7 @@
 	<modelVersion>4.0.0</modelVersion>
 	<groupId>br.gov.ans</groupId>
 	<artifactId>sei-broker</artifactId>
-	<version>2.6</version>
+	<version>2.7</version>
 	<packaging>war</packaging>
 	<name>sei-broker</name>
 	<description>Camada de integração REST para comunicação com o SEI</description>
@@ -39,6 +39,7 @@ import br.gov.ans.integracao.sei.modelo.enums.Acao;
 import br.gov.ans.integracao.sei.modelo.enums.TipoContato;
 import br.gov.ans.integracao.sei.utils.Constantes;
 import br.gov.ans.integracao.sei.utils.ContatoHelper;
+import br.gov.ans.integracao.sei.utils.MessagesKeys;
 import br.gov.ans.integracao.sei.utils.PessoaHelper;
 import br.gov.ans.utils.MessageUtils;
@@ -288,7 +289,7 @@ public class ContatoResource {
 				null, null, sigla, null, null, null, null, null);
     	if(ArrayUtils.isEmpty(contatos)){
-    		throw new ResourceNotFoundException(messages.getMessage("erro.contato.nao.encontrado",sigla));
+    		throw new ResourceNotFoundException(messages.getMessage(MessagesKeys.ERRO_CONTATO_NAO_ENCONTRADO, sigla));
     	}
     	return pessoaHelper.buildPessoa(contatos[0]);
@@ -62,6 +62,7 @@ import br.gov.ans.integracao.sei.modelo.ExclusaoDocumento;
 import br.gov.ans.integracao.sei.modelo.InclusaoDocumento;
 import br.gov.ans.integracao.sei.modelo.Operacao;
 import br.gov.ans.integracao.sei.utils.Constantes;
+import br.gov.ans.integracao.sei.utils.MessagesKeys;
 import br.gov.ans.utils.MessageUtils;
 import br.gov.ans.utils.MustacheUtils;
@@ -74,11 +75,9 @@ public class DocumentoResource {
     @Inject
     private UnidadeResource unidadeResource;
-    @SuppressWarnings("cdi-ambiguous-dependency")
     @Inject 
     private DAO<InclusaoDocumento> daoInclusaoDocumento;
-    @SuppressWarnings("cdi-ambiguous-dependency")
     @Inject 
     private DAO<ExclusaoDocumento> daoExclusaoDocumento;
@@ -517,7 +516,7 @@ public class DocumentoResource {
 		URL url = new URL(linkAcesso);
 		URLConnection con = url.openConnection();
-		String contentType = con.getHeaderField("Content-Type");
+		String contentType = con.getHeaderField(Constantes.CONTENT_TYPE_HEADER_KEY);
 		InputStream in = con.getInputStream();
@@ -669,7 +668,8 @@ public class DocumentoResource {
 			logger.debug(messages.getMessage("hash.em.geracao"));
 			return HashUtils.toSHA256(input, null);
 		} catch (Exception ex) {
-			logger.error(messages.getMessage("erro.calculo.hash"), ex);
+			logger.error(messages.getMessage("erro.calculo.hash"));
+			logger.debug(ex, ex);
 			return messages.getMessage("erro.calculo.hash");
 		}finally{
 			logger.debug(messages.getMessage("hash.gerado"));
@@ -1,6 +0,0 @@
-package br.gov.ans.integracao.sei.rest;
-
-public class MessagesKeys {
-	public static final String DEBUG_NOVO_DOCUMENTO_ENVIADO = "debug.novo.documento.enviado";
-	public static final String DEBUG_NOVO_DOCUMENTO_PROCESSADO = "debug.novo.documento.processado";
-}
@@ -254,7 +254,8 @@ public class ProcessoResource {
 					getSOuN(exibirInteressados), getSOuN(exibirObservacoes), getSOuN(exibirAndamento), getSOuN(exibirAndamentoConclusao), getSOuN(exibirUltimoAndamento), getSOuN(exibirUnidadesAberto), 
 					getSOuN(exibirProcessosRelacionados), getSOuN(exibirProcessosAnexados));
 		}catch(AxisFault ex){
-			logger.error(ex, ex);
+			logger.error(ex);
+			logger.debug(ex, ex);
 			return null;
 		}
 	}
@@ -0,0 +1,7 @@
+package br.gov.ans.integracao.sei.utils;
+
+public class MessagesKeys {
+	public static final String DEBUG_NOVO_DOCUMENTO_ENVIADO = "debug.novo.documento.enviado";
+	public static final String DEBUG_NOVO_DOCUMENTO_PROCESSADO = "debug.novo.documento.processado";
+	public static final String ERRO_CONTATO_NAO_ENCONTRADO = "erro.contato.nao.encontrado";
+}
	@@ -4,7 +4,7 @@		@@ -4,7 +4,7 @@
4	<modelVersion>4.0.0</modelVersion>	4	<modelVersion>4.0.0</modelVersion>
5	<groupId>br.gov.ans</groupId>	5	<groupId>br.gov.ans</groupId>
6	<artifactId>sei-broker</artifactId>	6	<artifactId>sei-broker</artifactId>
7	- <version>2.6</version>	7	+ <version>2.7</version>
8	<packaging>war</packaging>	8	<packaging>war</packaging>
9	<name>sei-broker</name>	9	<name>sei-broker</name>
10	<description>Camada de integração REST para comunicação com o SEI</description>	10	<description>Camada de integração REST para comunicação com o SEI</description>
	@@ -1,6 +0,0 @@	@@ -1,6 +0,0 @@
1	-package br.gov.ans.integracao.sei.rest;
2	-
3	-public class MessagesKeys {
4	- public static final String DEBUG_NOVO_DOCUMENTO_ENVIADO = "debug.novo.documento.enviado";
5	- public static final String DEBUG_NOVO_DOCUMENTO_PROCESSADO = "debug.novo.documento.processado";
6	-}
@@ -0,0 +1,7 @@		@@ -0,0 +1,7 @@
	1	+package br.gov.ans.integracao.sei.utils;
	2	+
	3	+public class MessagesKeys {
	4	+ public static final String DEBUG_NOVO_DOCUMENTO_ENVIADO = "debug.novo.documento.enviado";
	5	+ public static final String DEBUG_NOVO_DOCUMENTO_PROCESSADO = "debug.novo.documento.processado";
	6	+ public static final String ERRO_CONTATO_NAO_ENCONTRADO = "erro.contato.nao.encontrado";
	7	+}