Skip to content

Conversation

@adamchol
Copy link
Contributor

Implements proper whitelisting - before parsing anything.

@yeoleobun
Copy link
Collaborator

Good idea. If we don't need to update the whitelist at runtime, we should consider remove the Mutex and set the whitelist during initialization. If no whitelist is provided, we can default to an 'allow-all' white list.

@adamchol
Copy link
Contributor Author

adamchol commented Jan 28, 2026

Do you mean like a second field or some enum for a static whitelist? I do need a dynamic whitelist modified at runtime, that's why I did it this way.

@yeoleobun
Copy link
Collaborator

If dynamic loading is required, this implementation is reasonable. Thanks for the context.

@yeoleobun yeoleobun merged commit 7d66ad2 into restsend:main Jan 29, 2026
3 checks passed
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