Skip to content

Conversation

@ironirc
Copy link

@ironirc ironirc commented Oct 30, 2019

No description provided.

@noha
Copy link
Owner

noha commented Oct 31, 2019

Thanks! Where are the LcRSA and companion classes? With the PR we add a dependency which is not resolved

@ironirc
Copy link
Author

ironirc commented Nov 1, 2019

LcRSA stems from https://github.com/PierceNg/OpenSSL-Pharo
Is there a way to explicitly declare this dependency in the code or package definition?

Please note that this verification is done using the public key.
That's the reason I had to refactor a bit, instead of defining the signMessage method...

@ironirc
Copy link
Author

ironirc commented Dec 6, 2019

Thanks! Where are the LcRSA and companion classes? With the PR we add a dependency which is not resolved

I see the "resolve conflicts" pop up. Am I supposed to take action on this?

@noha
Copy link
Owner

noha commented Feb 16, 2020

Please have a look at #10 where your changes are incorporated

@noha noha closed this Feb 16, 2020
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.

2 participants