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.

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

Allow SASL_EXTERNAL to be a multiline string or a path #429

Open
arthsmn opened this issue Jan 8, 2024 · 4 comments
Open

Allow SASL_EXTERNAL to be a multiline string or a path #429

arthsmn opened this issue Jan 8, 2024 · 4 comments
Labels
feature request

Comments

@arthsmn
Copy link

arthsmn commented Jan 8, 2024

This would allow you to have your certificate in the config instead of a path.
@osa1
Copy link
Owner

osa1 commented Jan 9, 2024

cc resident SASL external expert @trevarj.

@arthsmn
Copy link
Author

arthsmn commented Jan 9, 2024

Also, it would be nice to allow home as a tilde and environment variables in the path.

@osa1
Copy link
Owner

osa1 commented Jan 9, 2024

Also, it would be nice to allow home as a tilde and environment variables in the path.

This was previously requested in #192. I would be happy to review and merge if someone implements it.

@trevarj
Copy link
Contributor

trevarj commented Jan 9, 2024

Seems reasonable. We can change the pem field to be a String here and then parse accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@osa1 @trevarj @arthsmn