Skip to content

Conversation

@juliusknorr
Copy link
Member

This will cause issues since the theming name can contain characters
that are not allowed in the local part of the mail address (like spaces).

This can still be configured with a proper mail address using mail_from_address / mail_domain in config.php

This will cause issues since the theming name can contain characters
that are not allowed in the local part of the mail address (like spaces)

Signed-off-by: Julius Härtl <[email protected]>
@juliusknorr juliusknorr added bug 3. to review Waiting for reviews labels Mar 12, 2020
@juliusknorr juliusknorr added this to the Nextcloud 19 milestone Mar 12, 2020
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

yes makes sense!

@rullzer rullzer requested a review from kesselb March 13, 2020 08:44
@rullzer rullzer merged commit a54c4b6 into master Mar 13, 2020
@rullzer rullzer deleted the bugfix/noid/invalid-from-mail branch March 13, 2020 09:17
@nickvergessen
Copy link
Member

backport needed or was this only in master?

@juliusknorr
Copy link
Member Author

Ah no, we should backport to all maintained versions.

@juliusknorr
Copy link
Member Author

/backport to stable18

@juliusknorr
Copy link
Member Author

/backport to stable17

@juliusknorr
Copy link
Member Author

/backport to stable16

@backportbot-nextcloud
Copy link

backport to stable18 in #19924

@backportbot-nextcloud
Copy link

backport to stable17 in #19925

@backportbot-nextcloud
Copy link

backport to stable16 in #19926

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants