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

Check index to avoid slice bounds out of range panic #475

Merged
merged 1 commit into from
Jun 17, 2020
Merged

Check index to avoid slice bounds out of range panic #475

merged 1 commit into from
Jun 17, 2020

Conversation

kiyonlin
Copy link
Contributor

It may raise panic when the hostname does not even contain one dot.

@kiyonlin kiyonlin requested a review from a team as a code owner June 17, 2020 03:08
@kiyonlin kiyonlin requested review from koddr and thomasvvugt and removed request for a team June 17, 2020 03:09
@welcome
Copy link

welcome bot commented Jun 17, 2020

Thanks for opening this pull request! 🎉 Please check out our contributing guidelines. If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

@Fenny Fenny merged commit 7677589 into gofiber:master Jun 17, 2020
@welcome
Copy link

welcome bot commented Jun 17, 2020

Congrats on merging your first pull request! 🎉 We here at Fiber are proud of you! If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

@Fenny Fenny mentioned this pull request Jun 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants