Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add implicit '/' to directories #22

Open
antonu17 opened this issue Feb 6, 2019 · 2 comments · May be fixed by #23
Open

Add implicit '/' to directories #22

antonu17 opened this issue Feb 6, 2019 · 2 comments · May be fixed by #23
Assignees
@pottava

Comments

@antonu17
Copy link

antonu17 commented Feb 6, 2019

Feature or enhancement request

Given s3 bucket content:

├── folder
│   └── index.html
└── index.html

Desired result:
GET / returns /index.html content
GET /folder/ returns /folder/index.html content
GET /folder returns /folder/index.html content

Actual result:
GET / returns /index.html content
GET /folder/ returns /folder/index.html content
GET /folder returns 404
antonu17 added a commit to antonu17/aws-s3-proxy that referenced this issue Feb 6, 2019
@antonu17
Add implicit '/' to directories
31c9331 
Something like this to fix pottava#22
@antonu17 antonu17 linked a pull request Feb 6, 2019 that will close this issue
Open
@pottava pottava self-assigned this Feb 7, 2019
@pottava
Copy link
Owner

pottava commented Feb 7, 2019

@antonu17 Thank you for your great opinion. It should be helpful for almost all projects. Let me see your PR later, thanks again :)

@e-k-m
Copy link

e-k-m commented Aug 18, 2020 •
edited
Loading

+ 1

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

@pottava pottava

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add implicit '/' to directories antonu17/aws-s3-proxy
3 participants
@pottava @e-k-m @antonu17