Skip to content
This repository has been archived by the owner on Dec 14, 2024. It is now read-only.

fix: Adding openssl as a vendored dependency #225

Merged
merged 7 commits into from
Sep 13, 2021

Conversation

daquino
Copy link
Contributor

@daquino daquino commented Sep 2, 2021

This PR includes the fix for the issue I mention here. It essentially is just adding openssl as a dependency.

Copy link
Member

@amaury1093 amaury1093 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works for me! Would you mind signing the CLA?

@CLAassistant
Copy link

CLAassistant commented Sep 13, 2021

CLA assistant check
All committers have signed the CLA.

@daquino daquino force-pushed the add-openssl-dependency branch from ca45054 to cdacfa0 Compare September 13, 2021 14:44
@daquino daquino force-pushed the add-openssl-dependency branch from cdacfa0 to 62905a1 Compare September 13, 2021 15:09
@daquino
Copy link
Contributor Author

daquino commented Sep 13, 2021

@AmauryM Alright, I've signed the CLA and updated the author info so it matches correctly. I haven't had to update author info for a past commit (usig git rebase) so it the commits look a little clunky. If you'd like me to just create a new PR with a nicer commit history, let me know.

@amaury1093 amaury1093 changed the title Adding openssl as a dependency. fix: Adding openssl as a vendored dependency Sep 13, 2021
@amaury1093
Copy link
Member

No worries, that's fine!

@amaury1093 amaury1093 merged commit 02255d7 into reacherhq:master Sep 13, 2021
@daquino daquino deleted the add-openssl-dependency branch September 13, 2021 21:15
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

3 participants