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

Resolve container versions and OpenAPI spec links through host-meta.json (RFC-6415) #3

Merged
merged 1 commit into from Apr 16, 2019

Conversation

schrieveslaach
Copy link
Contributor

When requesting the list of apps, PREvant tries to resolve .well-known/host-meta.json for each service which should provide versioning information and the link to the OpenAPI specification.

…son (RFC-6415)

When requesting the list of apps, PREvant tries to resolve .well-known/host-meta.json
for each service which should provide versioning information and the link to the OpenAPI
specification.
@schrieveslaach schrieveslaach requested a review from x1B April 9, 2019 15:17
Copy link
Contributor

@x1B x1B left a comment

Choose a reason for hiding this comment

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

Not sure I understand all the details, but LGTM.
I added two comments where something came to mind, but nothing that needs to be addressed within this PR.

api/src/models/service.rs Show resolved Hide resolved
)
.header(
"X-Forwarded-Prefix",
format!("/{}/{}", app_name, service_name),
Copy link
Contributor

Choose a reason for hiding this comment

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

One might add recursive Proxy support here (prepending any incoming X-Forwarded-Prefix).
Maybe this is premature, but somehow I get the feeling it'll be useful sooner or later.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you think of scenarios in which PREvant is running behind another proxy?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If so, do you want to add an issue so that we can keep track of it?

Copy link
Contributor

Choose a reason for hiding this comment

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

Good call, I opened #4 for this

@schrieveslaach schrieveslaach merged commit a85eed3 into master Apr 16, 2019
@schrieveslaach schrieveslaach deleted the feature/host-meta.json branch April 16, 2019 10:32
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.

None yet

2 participants