nacx commented on this pull request.
> +import com.google.inject.TypeLiteral; +import java.io.ByteArrayInputStream; +import java.util.logging.Level; +import java.util.logging.Logger; +import java.util.zip.ZipInputStream; +import javax.inject.Inject; +import javax.inject.Singleton; +import org.apache.jclouds.oneandone.rest.domain.VPNConfig; +import org.jclouds.http.functions.ParseJson; +import org.jclouds.json.Json; + +@Singleton +public class VPNConfigParser extends ParseJson<VPNConfig> { + + @Inject + public VPNConfigParser(Json json) { The visibility of this constructor still needs to be reduced by removing the public modifier (see [this](https://github.com/google/guice/wiki/KeepConstructorsHidden)). Or you can completely remove the parser class and bring back the `@SelectJson` annotation in the api method. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/319