Skip to content

[Mailer][Mime] Refactor S/MIME encryption handling in `SMimeEncryptionL… #20827

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.

Sign up for GitHub

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

Jump to bottom
Open
fabpot opened this issue Mar 26, 2025 · 2 comments · May be fixed by #20935
Open

[Mailer][Mime] Refactor S/MIME encryption handling in `SMimeEncryptionL… #20827

fabpot opened this issue Mar 26, 2025 · 2 comments · May be fixed by #20935
Labels
hasPR A Pull Request has already been submitted for this issue. Mailer Mime
Milestone
7.3

Comments

@fabpot
Copy link
Member

fabpot commented Mar 26, 2025

Q A
Feature PR symfony/symfony#59831
PR author(s) @Spomky
Merged in 7.3

We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.

To fix this issue, please create a PR against the 7.3 branch in the symfony-docs repository.

Thank you! 😃
@fabpot fabpot added this to the 7.3 milestone Mar 26, 2025
@alamirault
Copy link
Contributor

@Spomky What is the expected value for repository ? Description is Path to the S/MIME certificate repository. Shall implement the Symfony\Component\Mailer\EventListener\SmimeCertificateRepositoryInterface. Isn't a service id inseated ? I didnt' see real example in PR

Image

Image

Documentation to update: d87be4e

@Spomky
Copy link
Contributor

Spomky commented May 2, 2025

Hello @alamirault,

Thanks for the good remark. I opened symfony/symfony#60335 to clarify that a service is expected.
You're right. The PR description didn’t contain any example. I just added one.
Let me know if you have any other question or feedback.

@alamirault alamirault linked a pull request May 4, 2025 that will close this issue
Open
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue. Mailer Mime
Projects
None yet
Milestone
7.3
5 participants
@fabpot @javiereguiluz @Spomky @alamirault @carsonbot