Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate forbiddenAPIs checks into ElasticSearch #3059

Closed
s1monw opened this Issue May 18, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@s1monw
Copy link
Contributor

s1monw commented May 18, 2013

https://code.google.com/p/forbidden-apis/ integrates with maven and checks Java byte code against a list of "forbidden" API signatures. It fails the builds if deprecated JDK methods are used or if Streams are created with default charsets, if default locales or timezones are used or if encodings are missing on String formatting etc.

This helps to prevent issues that occur on the users systems if locals are different or charsets are oddly set.

@ghost ghost assigned s1monw May 18, 2013

@uschindler

This comment has been minimized.

Copy link
Contributor

uschindler commented May 18, 2013

I added a comment to your commit :-)

@s1monw s1monw closed this in 31f0aca May 19, 2013

s1monw added a commit that referenced this issue May 19, 2013

Integrate forbiddenAPI checks into Maven build.
This commit integrates the forbiddenAPI checks that checks
Java byte code against a list of "forbidden" API signatures.
The commit also contains the fixes of the current source code
that didn't pass the default API checks.

See https://code.google.com/p/forbidden-apis/ for details.

Closes #3059

mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015

Integrate forbiddenAPI checks into Maven build.
This commit integrates the forbiddenAPI checks that checks
Java byte code against a list of "forbidden" API signatures.
The commit also contains the fixes of the current source code
that didn't pass the default API checks.

See https://code.google.com/p/forbidden-apis/ for details.

Closes elastic#3059
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.