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

Feat/cse gmail api #243

Merged
merged 9 commits into from
May 2, 2024
Merged

Feat/cse gmail api #243

merged 9 commits into from
May 2, 2024

Conversation

ccorsin
Copy link
Contributor

@ccorsin ccorsin commented Apr 26, 2024

No description provided.

@ccorsin ccorsin force-pushed the feat/cse_gmail_api branch 3 times, most recently from 90e4a2b to 23e7cb6 Compare April 29, 2024 09:47
@ccorsin ccorsin changed the title Draft: Feat/cse gmail api Feat/cse gmail api Apr 29, 2024
@ccorsin ccorsin force-pushed the feat/cse_gmail_api branch 6 times, most recently from e47dc0d to 6f54e62 Compare April 30, 2024 10:09
@ccorsin ccorsin requested review from tbrezot and Manuthor April 30, 2024 10:10
Copy link
Contributor

@Manuthor Manuthor left a comment

Choose a reason for hiding this comment

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

Great job!!
You could update the autogenerated documentation/docs/cli/main_commands.md.
Otherwise I know we are waiting for a work around the Certify operation to be able to generate RSA keys (using specific Openssl extension file): once it will be done, we should get rid off all the p7pem and wrap files and use keys only from memory. I guess we can merge this PR and create a new one later.

Copy link
Contributor

@tbrezot tbrezot left a comment

Choose a reason for hiding this comment

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

Nice job: be careful to needless copies, otherwise some nice things!

@ccorsin ccorsin force-pushed the feat/cse_gmail_api branch 3 times, most recently from 467b292 to 9b00556 Compare May 2, 2024 13:01
@ccorsin ccorsin force-pushed the feat/cse_gmail_api branch from 9b00556 to 3561c62 Compare May 2, 2024 13:51
@ccorsin ccorsin merged commit 656a648 into develop May 2, 2024
33 checks passed
@ccorsin ccorsin deleted the feat/cse_gmail_api branch May 2, 2024 14:28
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.

3 participants