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

fix(file) populate port according to protocol in url #166

Merged
merged 1 commit into from
Jun 8, 2020

Conversation

hbagdi
Copy link
Member

@hbagdi hbagdi commented Jun 3, 2020

Populate port according to the protocol scheme in the url sugar
attribute. Default port of 80 is now populated to match Kong's
behavior in the same case.

@hbagdi
Copy link
Member Author

hbagdi commented Jun 3, 2020

cc @rainest

Populate port according to the protocol scheme in the `url` sugar
attribute. Default port of `80` is now populated to match Kong's
behavior in the same case.
@hbagdi hbagdi force-pushed the fix/default-port-in-url branch from ec9bc21 to 83881a6 Compare June 8, 2020 18:41
@hbagdi hbagdi merged commit 83fa36f into master Jun 8, 2020
@hbagdi hbagdi deleted the fix/default-port-in-url branch June 8, 2020 18:42
rainest pushed a commit that referenced this pull request Apr 21, 2021
Populate port according to the protocol scheme in the `url` sugar
attribute. Default port of `80` is now populated to match Kong's
behavior in the same case.

From #166
AntoineJac pushed a commit that referenced this pull request Jan 23, 2024
Populate port according to the protocol scheme in the `url` sugar
attribute. Default port of `80` is now populated to match Kong's
behavior in the same case.

From #166
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.

1 participant