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

Redirect if request path is directory #19

Closed
srngch opened this issue Oct 9, 2022 · 0 comments · Fixed by #36
Closed

Redirect if request path is directory #19

srngch opened this issue Oct 9, 2022 · 0 comments · Fixed by #36
Assignees
@hhkim0729 @srngch
Labels
feature New feature or request

Comments

@srngch
Copy link
Owner

srngch commented Oct 9, 2022 •
edited
Loading

  • URI가 /directory 일 경우 -> /directory/ 로 301 redirect 응답 보내기
@hhkim0729 hhkim0729 added the feature New feature or request label Oct 14, 2022
hhkim0729 added a commit that referenced this issue Oct 16, 2022
@hhkim0729 @srngch
feat: Implement directory path redirection(#19)
986cd4d 
Additional tasks
- Add 301 Moved Permanently macro
- Move filePath, originalHeader, originalBody member variables from Worker to Request
- Add appendHeader() to Response

Co-authored-by: srngch <srngch.dev@gmail.com>
@hhkim0729 hhkim0729 mentioned this issue Oct 16, 2022
Merged
hhkim0729 added a commit that referenced this issue Oct 16, 2022
@hhkim0729
feat: Change redirect location to relative path(#19)
689e76c
hhkim0729 added a commit that referenced this issue Oct 16, 2022
@hhkim0729
Merge pull request #36 from srngch/feature/19/redirect-directory
8e15bce 
feat: Implement directory path redirection(#19)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hhkim0729 hhkim0729

@srngch srngch

Labels
feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Implement directory path redirection(#19) srngch/ngin-xs
2 participants
@hhkim0729 @srngch