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

Support for relative URLs in Location header #23385

Open
thomas11 opened this issue Aug 29, 2024 · 2 comments
Open

Support for relative URLs in Location header #23385

thomas11 opened this issue Aug 29, 2024 · 2 comments
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@thomas11
Copy link

Feature Request

The HTTP Semantics RFC shows that URLs in the Location header can be relative, and the client should turn them into absolute URLs by adding the relevant path:

For example, a GET request generated for the URI reference "http://www.example.org/~tim" might result in a 303 (See Other) response containing the header field:
Location: /People.html#tim
which suggests that the user agent redirect to "http://www.example.org/People.html#tim"

Where as the poller implementation explicitly asserts that the URL is absolute.

To fully support the RFC, handling relative Locations would be required.

I ran into this issue once with an Azure service returning a relative Location, but unfortunately didn't capture the error at the time.

@github-actions github-actions bot added customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Aug 29, 2024
@jhendrixMSFT
Copy link
Member

Do you know which Azure service returned the relative URL? For ARM services, it's required that URLs for polling are absolute (see Azure RPC spec).

@thomas11
Copy link
Author

I don't, unfortunately. I'll see if I can repro it again.

Good to know that absolute URLs are required, so the relative URL - if I didn't mess up on my part - was a rare exception.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

2 participants