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

If data is not provided, extract it from the PKCS7 instance. #133

Merged
merged 1 commit into from
May 4, 2017

Conversation

headius
Copy link
Member

@headius headius commented May 3, 2017

Fixes #132.

@kares I'm not sure if there's an MRI test that would pass now because of this change, or if we should just add our own (and not sure where that would go).

@kares
Copy link
Member

kares commented May 4, 2017

great, we can add a test later (mostly use our own suite for these). feel free to merge or I will later, thanks.

@headius
Copy link
Member Author

headius commented May 4, 2017

Ok thanks for the clarification.

@headius headius merged commit 2b883bd into jruby:master May 4, 2017
@headius headius deleted the fix-132 branch May 4, 2017 13:51
@headius headius added this to the 0.9.21 milestone May 4, 2017
@headius
Copy link
Member Author

headius commented May 4, 2017

@kares Milestones seem to have not been updated since 0.9.18, so I closed out 0.9.18-0.9.20 and added 21 and 22, but I don't know for certain which issues were in which releases. I think we should try to have every issue tagged with a release, as in JRuby.

#132 and #133 have been targeted for 0.9.21.

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

Successfully merging this pull request may close these issues.

None yet

2 participants