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

Generalise SwaggerUI plug config supplementing for Phoenix app #547

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ycherniavskyi
Copy link
Contributor

Resolve #546.

The previous implementation of the SwaggerUI plug's configuration supplementing function for the Phoenix app was limited. It only applied to oauth2_redirect_url and joined the path only with the Endpoint URL. This approach was inaccurate if a path was already present in the Endpoint URL configuration.

This PR enhances the configuration supplementing functionality. It introduces the ability to join desired paths with :endpoint_url and :endpoint_path. This flexibility is provided for all configuration values that adhere to the respective structure.

@ycherniavskyi
Copy link
Contributor Author

It appears there's a problem with the CI checks. This issue seems to be associated with the runtime of the Endpoint and its corresponding configuration. Investigating...

@ycherniavskyi
Copy link
Contributor Author

@zorbash, apologies for reaching out directly, but I noticed you managed the last release of this project. Could you assist with reviewing this PR, or direct me to the appropriate person to contact?

If this PR needs further updates, expansions, or adjustments to the code, tests, or documentation, please let me know, and I'll make the necessary changes.

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.

SwaggerUI Plug ignores Phoenix Endpoint url configuration
2 participants