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

V9: Error when you try to invite user via backoffice #11453

Closed
Aaen opened this issue Oct 22, 2021 · 4 comments
Closed

V9: Error when you try to invite user via backoffice #11453

Aaen opened this issue Oct 22, 2021 · 4 comments

Comments

@Aaen
Copy link
Contributor

Aaen commented Oct 22, 2021

Which exact Umbraco version are you using? For example: 8.13.1 - don't just write v8

This is on Umbraco 9.0.1 Umbraco Cloud

Bug summary

When you try to invite a user from the backoffice you will get an error I have included the error in the specifics

Specifics

Skærmbillede 2021-10-22 kl  09 57 14

Unfortunately there is no error in the logviewer for this one

Steps to reproduce

  1. Go tot the backoffice
  2. Go to the user section
  3. Click invite user
  4. Follow the flow for this and click invite user

Expected result / actual result

I would expect that I can invite users to the backoffice without any issues


This item has been added to our backlog AB#15057

@Aaen Aaen added the type/bug label Oct 22, 2021
@bergmania bergmania added state/sprint-candidate We're trying to get this in a sprint at HQ in the next few weeks project/v9 labels Oct 22, 2021
@bjarnef
Copy link
Contributor

bjarnef commented Nov 4, 2021

Oh I just noticed this as well also v9.0.1 🙈
It would be great to get this fixed as we soon need to invite content editors, but currently we need to create users manually or enabled the invited user afterwards.

@ronaldbarendse
Copy link
Contributor

Looks like the error occurs because 9.0.1 doesn't set the sender address to the back-office user that's sending the invite anymore (which was a bug, as that might not be a valid sender address for the configured SMTP settings): #11266.

Even though Umbraco Cloud adds a fallback for sending invitations without having SMTP configured, this doesn't expect the null value, hence the Value cannot be null error.

You can work-around this issue by at least providing the Umbraco:CMS:Global:Smtp:From setting as shown in the example on https://our.umbraco.com/documentation/umbraco-cloud/Set-up/SMTP-settings/. If you want to send out emails, you will need to configure the SMTP server anyway...

@bjarnef
Copy link
Contributor

bjarnef commented Nov 9, 2021

If you want to send out emails, you will need to configure the SMTP server anyway...

Yeah, except when the client don't use form or similar to send emails, but only need the feature to invite other users. Previous this required SMTP to be configured, but in a V8 version this changed, so the invite user button always was shown on Umbraco Cloud environments and thus didn't required further SMTP configuration to use the invite user flow.

Not sure if it is related to this: #11575

@bjarnef
Copy link
Contributor

bjarnef commented Nov 14, 2021

@ronaldbarendse it doesn't seem to be enough just providing the Umbraco:CMS:Global:Smtp:From setting.

It still throw this exception:

image

False MSAL 4.35.1.0 MSAL.NetCore .NET 5.0.9 Microsoft Windows 10.0.14393 [11/14 14:20:05.38 - 696e8355-dae8-4ab7-a6e4-132a6a82d505] Exception type: Microsoft.Identity.Client.MsalUiRequiredException , ErrorCode: user_null HTTP StatusCode 0 CorrelationId at Microsoft.Identity.Client.Internal.Requests.Silent.SilentRequest.ExecuteAsync(CancellationToken cancellationToken) at Microsoft.Identity.Client.Internal.Requests.Silent.SilentRequest.ExecuteAsync(CancellationToken cancellationToken) at Microsoft.Identity.Client.Internal.Requests.RequestBase.RunAsync(CancellationToken cancellationToken)

Update: Okay, it actually does seem to work now. Not sure if something was cached on production/live for some time.

@elit0451 elit0451 removed the state/sprint-candidate We're trying to get this in a sprint at HQ in the next few weeks label Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants