Skip to content

If username_domain has only one domain, don't allow users to specify the...#3

Closed
corretge wants to merge 1 commit intoroundcube:masterfrom
corretge:patch-1
Closed

If username_domain has only one domain, don't allow users to specify the...#3
corretge wants to merge 1 commit intoroundcube:masterfrom
corretge:patch-1

Conversation

@corretge
Copy link
Copy Markdown

... doamin.

i.e. Google Apps domains connect to the same imap server

…the doamin.

i.e. Google Apps domains connect to the same imap server
@thomascube
Copy link
Copy Markdown
Member

Hi Alex

Thanks for your pull request! The idea sonds reasonable but I'd rather use a specific config parameter (e.g. force_domain) instead of changing the behavior of the existing option 'username_domain'.

@corretge
Copy link
Copy Markdown
Author

Hi Thomas, I change the script quickly to prevent that all gmail users could create accounts in my server :)
I edited this script from the github editor, and create the fork and pathc (awesome)
Thanks for consider this configuration option and for the great job in Roundcube.

@thomascube
Copy link
Copy Markdown
Member

We'll accept a new pull request with a new config option like 'force_domain' for this.

@thomascube thomascube closed this Jul 17, 2013
@alecpl alecpl mentioned this pull request Sep 4, 2017
@be-root be-root mentioned this pull request Mar 24, 2021
@alxbr alxbr mentioned this pull request Dec 12, 2021
@amend07 amend07 mentioned this pull request Jun 6, 2024
2 tasks
@Ressy66 Ressy66 mentioned this pull request Jun 17, 2025
2 tasks
@dsvf dsvf mentioned this pull request Aug 17, 2025
2 tasks
@wdsnyc wdsnyc mentioned this pull request Nov 28, 2025
@joksik joksik mentioned this pull request Feb 27, 2026
2 tasks
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.

2 participants