Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Update certificate parsing for certificate chains with multiple certificates #50

Conversation

jakul
Copy link
Contributor

@jakul jakul commented Jul 18, 2016

Fix for #49.

Update the certificate parsing so that it works when certificate chains have multiple certificates within them.

@carlnordenfelt Can you review & merge please?

@carlnordenfelt
Copy link
Owner

Hi,

I'll take a look at the PR as soon as I get back to civilization which is
on thursday.

On Mon, Jul 18, 2016, 16:47 Craig Blaszczyk notifications@github.com
wrote:

Fix for #49
#49
.

Update the certificate parsing so that it works when certificate chains
have multiple certificates within them.

@carlnordenfelt https://github.com/carlnordenfelt Can you review &

merge please?

You can view, comment on, or merge this pull request online at:

#50
Commit Summary

  • Update certificate parsing for certificate chains with multiple
    certificates

File Changes

Patch Links:

https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50.patch

https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50.diff


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#50,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEzRoZTI7ZdQ-MuZxQjHSAjhozAmSW1kks5qW5HUgaJpZM4JOyi_
.

@carlnordenfelt carlnordenfelt merged commit 0f802a2 into carlnordenfelt:master Jul 20, 2016
@carlnordenfelt
Copy link
Owner

I've merged but won't be able to relate a new package until tomorrow. In the meantime you can create your own package and deploy if you want.

@carlnordenfelt
Copy link
Owner

2.0.1 has been released with the fix.

@jakul
Copy link
Contributor Author

jakul commented Jul 22, 2016

Cheers!

Em qui, 21 de jul de 2016 17:21, Carl Nordenfelt notifications@github.com
escreveu:

2.0.1 has been released with the fix.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#50 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0qSmR4MoOvgWhPumX8zbfPgzFZcQJFks5qX5yigaJpZM4JOyi_
.

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

Successfully merging this pull request may close these issues.

None yet

2 participants