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

Restore possibility to define multiple URL per backend #347

Open
MPStudyly opened this issue Oct 11, 2022 · 1 comment · May be fixed by #770
Open

Restore possibility to define multiple URL per backend #347

MPStudyly opened this issue Oct 11, 2022 · 1 comment · May be fixed by #770
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@MPStudyly
Copy link

MPStudyly commented Oct 11, 2022

Hello everyone,

I just encountered an issue while helping to set up a spreed instance for a Nextcloud instance which is reachable via regular A (and AAAA) records, but also via a convenience (shorter) CNAME record.
When accessing the server via the A record, everything works fine and a connection to the signalling server can be established, but when connecting via the CNAME record it obviously fails, due to that URL missing in the signaling config. I noticed that before #28 was implemented, it was possible to specify multiple URLs within the "allowed" property. Since that change it is not possible to set multiple URLs anymore, which is required, when a single Nextcloud instance is reachable via multiple domains (regardless of DNS record types), such as in our scenario.
So my request is to re-allow specification of multiple URLs per backend, or, if that is not feasible, to allow communication between multiple backends when explicitly enabled via config toggle or by defining some sort of mapping (e.g. backend a can talk to b,c,d and vice versa, e can talk to f,g, etc.).

Best Regards
Michael

@fancycode fancycode added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Jan 17, 2023
@fancycode
Copy link
Member

This is currently not planned, but I'll happily accept pull-requests that implement multiple URLs per backend.

@fancycode fancycode linked a pull request Jul 3, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants