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

JRuby failures on master since v1.6.2 #448

Closed
christopher-gordon opened this issue Mar 5, 2018 · 7 comments
Closed

JRuby failures on master since v1.6.2 #448

christopher-gordon opened this issue Mar 5, 2018 · 7 comments

Comments

@christopher-gordon
Copy link

Apologies if this overlaps with another issue/PR. It appears the JRuby tests have been failing on master since v1.6.2.

I'm unfortunately unable to help debug the issue, but I'm hoping someone else is able to spot the problem easily. I'm attempting to upgrade from v1.3.1 to a version that is working in JRuby that also includes the fix for CVE-2017-11428.

Thanks for your help.

@pitbulk
Copy link
Collaborator

pitbulk commented Mar 5, 2018

I don't have a Jruby environment right now to debug.

Related to one of the errors, Java::JavaLang::NullPointerException: nid not found for oid = '0.0'
I found:

so maybe is a new regression?

Rest of the errors seems related to the decryption process.

@pitbulk
Copy link
Collaborator

pitbulk commented Mar 5, 2018

I noticed that

So something changed on the Travis environment

@pitbulk
Copy link
Collaborator

pitbulk commented Mar 6, 2018

@christopher-gordon Try to git clone the repository in a different path but using the same environment that your main project and try to execute the test there, I guess them will pass in your environment.

@tomilaine
Copy link
Contributor

I cloned the repo and run the tests locally with jruby 9.1.13.0. No errors.
(468 runs, 1211 assertions, 0 failures, 0 errors, 1 skips)

@pitbulk
Copy link
Collaborator

pitbulk commented Mar 6, 2018

Ty @tomilaine so is a Travis CI stuff.

@tomilaine
Copy link
Contributor

Tests pass locally for jruby 1.7.27 as well.

@christopher-gordon
Copy link
Author

Thanks for the quick responses on this, everyone. Sounds like this can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants