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.

Sign up for GitHub

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

Jump to bottom

Fix IPv6 localhost regex #24730

Merged
merged 1 commit into from
Dec 21, 2020
Merged

Fix IPv6 localhost regex #24730

merged 1 commit into from
Dec 21, 2020

Conversation

J0WI
Copy link
Contributor

@J0WI J0WI commented Dec 16, 2020 •
edited
Loading

STR:

  1. run a local nextcloud instance
  2. access on 127.0.0.1 => works
  3. access on [::1] => access denied, address not in trusted_domains config

The example in the docs is also using brackets.

@J0WI
Fix IPv6 localhost regex
331f30f 
Signed-off-by: J0WI <J0WI@users.noreply.github.com>
@J0WI J0WI added the 3. to review Waiting for reviews label Dec 16, 2020
@rullzer rullzer added this to the Nextcloud 21 milestone Dec 18, 2020
@rullzer rullzer mentioned this pull request Dec 18, 2020
Merged
39 tasks
@juliusknorr juliusknorr merged commit fbbb48f into nextcloud:master Dec 21, 2020
@J0WI J0WI deleted the fix-trusted-ipv6 branch December 22, 2020 20:52
@J0WI
Copy link
Contributor Author

J0WI commented Dec 22, 2020

/backport to stable20

@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request Dec 22, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rullzer rullzer rullzer approved these changes

@juliusknorr juliusknorr juliusknorr approved these changes

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

@kesselb kesselb Awaiting requested review from kesselb

Assignees
No one assigned
Labels
3. to review Waiting for reviews
Projects
None yet
Milestone
Nextcloud 21
Development

Successfully merging this pull request may close these issues.
3 participants
@J0WI @rullzer @juliusknorr