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

docs: option address, not hostname #777

Merged
merged 1 commit into from
Oct 3, 2021

Conversation

jasonprado
Copy link
Contributor

The field is address in the code, not hostname. Setting hostname has no effect.

See: https://github.com/jamhall/s3rver/blob/main/lib/s3rver.js#L146

@kherock kherock merged commit dee1be9 into jamhall:main Oct 3, 2021
@kherock
Copy link
Collaborator

kherock commented Oct 3, 2021

Thanks @jasonprado!

kherock pushed a commit that referenced this pull request Oct 3, 2021
Co-authored-by: Jason Prado <jason@drivers.coop>
@jasonprado jasonprado deleted the readme-address-not-hostname branch October 5, 2021 15:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants