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

refactor: force ipv4 localhost #22394

Merged
merged 2 commits into from
Sep 13, 2023
Merged

Conversation

0xsirsaif
Copy link
Contributor

Replacing localhost with 127.0.0.1 in the codebase; sometimes the name localhost force-resolves to ipv6

Replacing "localhost" with "127.0.0.1" in the codebase; sometimes the name localhost force-resolves to ipv6
@0xsirsaif 0xsirsaif requested review from a team and surajshetty3416 and removed request for a team September 13, 2023 10:05
@ankush ankush added the Run UI Tests Runs Server CI builds even if frontend changes not detected. label Sep 13, 2023
@ankush ankush added the backport version-14-hotfix backport to version 14 label Sep 13, 2023
@ankush ankush changed the title reafctor: force ipv4 localhost refactor: force ipv4 localhost Sep 13, 2023
@ankush ankush enabled auto-merge (squash) September 13, 2023 10:11
@ankush ankush removed the Run UI Tests Runs Server CI builds even if frontend changes not detected. label Sep 13, 2023
@ankush ankush merged commit aa7ee1c into frappe:develop Sep 13, 2023
20 of 21 checks passed
mergify bot pushed a commit that referenced this pull request Sep 13, 2023
* reafctor: force ipv4 localhost

Replacing "localhost" with "127.0.0.1" in the codebase; sometimes the name localhost force-resolves to ipv6

* revert: leave localhost usage in oauth tests

Change not required.

---------

Co-authored-by: Ankush Menat <ankush@frappe.io>
(cherry picked from commit aa7ee1c)
ankush pushed a commit that referenced this pull request Sep 13, 2023
* reafctor: force ipv4 localhost

Replacing "localhost" with "127.0.0.1" in the codebase; sometimes the name localhost force-resolves to ipv6

* revert: leave localhost usage in oauth tests

Change not required.

---------

Co-authored-by: Ankush Menat <ankush@frappe.io>
(cherry picked from commit aa7ee1c)

Co-authored-by: 0xsaif <muhamed.e.saif@gmail.com>
@0xsirsaif 0xsirsaif deleted the fix-localhost-renaming branch September 13, 2023 11:40
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants