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

Consider returning target URL in body on redirect #44

Open
onebytegone opened this issue Apr 2, 2019 · 1 comment
Open

Consider returning target URL in body on redirect #44

onebytegone opened this issue Apr 2, 2019 · 1 comment
Assignees
@onebytegone

Comments

@onebytegone
Copy link
Contributor

Since v1.0.0 (original commit), express will return the target URL in the body of a redirection response. Their current implementation is slightly different, but the concept still exists. This might be helpful functionality to add to lambda-express.
@onebytegone onebytegone changed the title Considering returning target URL in body on redirect Consider returning target URL in body on redirect Apr 2, 2019
onebytegone added a commit that referenced this issue Apr 2, 2019
@onebytegone
feat: Add plain text body for redirects (#44)
ebb18d3
jthomerson added a commit that referenced this issue Apr 2, 2019
@jthomerson
Merge pull request #45 from silvermine/fix_44_redirect_body
149d560 
feat: Add plain text body for redirects (#44)
@onebytegone
Copy link
Contributor Author

onebytegone commented Apr 2, 2019 •
edited
Loading

The initial work for this was handled in #45. However, there is still a TODO to address once Accept header parsing is implemented (See #46).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@onebytegone onebytegone

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@onebytegone