bazaarvoice/seo_sdk_java

Bazaarvoice SEO SDK for Java

Bazaarvoice SEO SDK for Java

This repository contains the codebase for the Bazaarvoice SEO SDK for Java, organized as a Maven project. For documentation, useful related links and binary releases, please visit Bazaarvoice Knowledge.

seo_sdk_java's Issues

AppDynamics reports errors for 404s for known behavior

Hello, we're using appdynamics and bazaarvoice at my organization. I happened to notice the huge numbers of errors reported by appdynamics because this line is failing on products with no reviews:

seo_sdk_java/src/main/java/com/bazaarvoice/seo/sdk/BVUIContentServiceProvider.java

Line 211 in 218d8a1

is = httpUrlConnection.getInputStream();

java.io.FileNotFoundException
http://seo.bazaarvoice.com/perfumania-47c02ed0cca1c657e83a0d7544d7a0d2/Main_Site-en_US/reviews/product/1/110000321.htm
at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1479)
at com.bazaarvoice.seo.sdk.BVUIContentServiceProvider.loadContentFromHttp(BVUIContentServiceProvider.java:211)
at com.bazaarvoice.seo.sdk.BVUIContentServiceProvider.getBvContent(BVUIContentServiceProvider.java:145)
at com.bazaarvoice.seo.sdk.BVUIContentServiceProvider.call(BVUIContentServiceProvider.java:85)
at com.bazaarvoice.seo.sdk.BVUIContentServiceProvider.call(BVUIContentServiceProvider.java:57)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:314)
at java.util.concurrent.FutureTask.run(FutureTask.java:149)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:895)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:918)
at java.lang.Thread.run(Thread.java:767)

Might want to catch that, since this is definitely a known behavior and not an exceptional behavior.

Thanks.

Push Artifact to Maven Central

Is it on your backlog to support distribution of this artifact to maven central? Or does it exist in some other public repository?

BVHTMLFooter is depending on log4j

The pom states that log4j is in scope test bug the BVHTMLFooter velocity template is Log4JLogChute and so requiring log4j. We had a lot of debug errors, our logs where over floating with the debug error:

[debug] Couldn't find class org.apache.velocity.runtime.log.Log4JLogChute or necessary supporting classes in classpath.
org/apache/log4j/Layout
java.lang.NoClassDefFoundError: org/apache/log4j/Layout
at java.lang.Class.forName0(Native Method)
at java.lang.Class.forName(Class.java:274)
at org.apache.velocity.util.ClassUtils.getClass(ClassUtils.java:63)
at org.apache.velocity.util.ClassUtils.getNewInstance(ClassUtils.java:95)
at org.apache.velocity.runtime.log.LogManager.createLogChute(LogManager.java:147)
at org.apache.velocity.runtime.log.LogManager.updateLog(LogManager.java:208)
at org.apache.velocity.runtime.RuntimeInstance.initializeLog(RuntimeInstance.java:728)
at org.apache.velocity.runtime.RuntimeInstance.init(RuntimeInstance.java:240)
at org.apache.velocity.runtime.RuntimeInstance.getTemplate(RuntimeInstance.java:1099)
at org.apache.velocity.runtime.RuntimeInstance.getTemplate(RuntimeInstance.java:1077)
at org.apache.velocity.app.VelocityEngine.getTemplate(VelocityEngine.java:528)
at com.bazaarvoice.seo.sdk.footer.BVHTMLFooter.displayFooter(BVHTMLFooter.java:92)

Because we don't have log4j on our classpath, we are using logback as the log framework, the sdk however depends on slf4j so why log4j is in the footer I don't know.

Kind regards,

Joris Vanleene.