Skip to content

reverse_proxy_configuration.rst: IPv6 CIDR is supported - #11231

Merged
kesselb merged 1 commit into
nextcloud:masterfrom
jivanpal:patch-1
Nov 21, 2023
Merged

reverse_proxy_configuration.rst: IPv6 CIDR is supported#11231
kesselb merged 1 commit into
nextcloud:masterfrom
jivanpal:patch-1

Conversation

@jivanpal

Copy link
Copy Markdown
Contributor

Confirmed that IPv6 CIDR notation works on server v27.0.0. Apparently, this has been supported since v25.0.0-beta1; see nextcloud/server#32615

…ince v25.0.0-beta1
Signed-off-by: Jivan Pal <jivan.pal@gmail.com>

@joshtrichardsjoshtrichards left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 And looks like the config sample got taken care of in nextcloud/server#35956.

@jivanpal

Copy link
Copy Markdown
ContributorAuthor

@joshtrichards Would we like to add the example addresses to match nextcloud/server#35956 ?

@tflidd

Copy link
Copy Markdown
Contributor

If you want to add an example, I'd rather use a subnet size of at least /64. But if you add it, wouldn't you need to put ipv4 examples as well?

@kesselb
kesselb merged commit 3c60313 into nextcloud:masterNov 21, 2023
@kyrofa

kyrofa commented Nov 27, 2023

Copy link
Copy Markdown
Member

This is so timely, thank you @jivanpal. I was just horrified to discover that ipv6 cidrs weren't supported, and started digging into why, only to learn that they are! The v27 docs are just out of date. @kesselb, should we backport this to other supported releases?

@joshtrichards

Copy link
Copy Markdown
Member

/backport to stable27

@joshtrichards

Copy link
Copy Markdown
Member

/backport to stable26

@joshtrichards

Copy link
Copy Markdown
Member

Done, @kyrofa. Thanks for the reminder!

@kyrofa

Copy link
Copy Markdown
Member

My pleasure, thank YOU!

@jivanpal
jivanpal deleted the patch-1 branch December 31, 2023 00:17
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@jivanpal@tflidd@kyrofa@joshtrichards@kesselb