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

[CW-1400] Getting Microsoft Email working with Chatwoot in 2023 #6729

Closed
evolve2k opened this issue Mar 22, 2023 · 10 comments
Closed

[CW-1400] Getting Microsoft Email working with Chatwoot in 2023 #6729

evolve2k opened this issue Mar 22, 2023 · 10 comments
Assignees
Labels
feature-request This label is used to track new features which is a significant change to existing product investigation This issues needs more work before it can be picked up for development linear

Comments

@evolve2k
Copy link

evolve2k commented Mar 22, 2023

As of 2023 basic auth is totally removed (following Microsofts deprecation notices in 2022).

Getting Microsoft Email has become complicated in 2023 and Microsoft arent helping things.
This issue is to address a range of issues that are making it hard to setup Microsoft email for use with Chatwoot.

  1. Microsoft Publisher Verification not required for Single Tenancy Access ("emails at one domain")

Until last week the guide implied (at least from the screenshots) to setup with multi-tenancy access when you register your Azure Application. If you only want to support email inboxes at a single domain, time-consuming organisational verification is not required. The guide being is updated to make 'single tenancy access' recommended, and to explain whats involved if you want multi-tenancy. This was approved recently, and as time of writing is now live! 🙌
chatwoot/docs#389

  1. Single tenancy access setup currently causes an error in the logs

Setup as single tenancy access generates an error in the server logs, like this:
[Your Application] is not configured as a multi-tenant application. Usage of the /common endpoint is not supported for such applications created after '10/15/2018'. Use a tenant-specific endpoint or configure the application to be multi-tenant.

This is occuring as currently chatwoot hardcodes a url endpoint that includes '/common' but for single tenancy access apps, this needs to be set to the project name.

We're glad to report, we've patched this ourselves and have this working, our devs unfamiliarity with your test suite has meant the PR has no tests currently, but we hope this makes it easy for someone to take it from here.

We have this working in #6728

  1. With endpoint fixed, Outlook emails setup correctly and can receive BUT issue sending.

While Microsoft said that SMTP can be used once Oath2.0 connectivity is setup (via registering your own Azure application), seems that this is not the case and SMTP remains broken for sending emails.

The issue here is that if not using a verified app, ALL SMTP functionality stops working and that equivilent functionality needs to be reimplemented using Micrsofts Graph API.

We have this working in #6728

We're looking for someone to add tests to the PR and to get it merged in.

CW-1400

@pranavrajs
Copy link
Member

@evolve2k Thanks for the detailed note.

ALL SMTP functionality stops working and that equivilent functionality needs to be reimplemented using Micrsofts Graph API.

Is it a consistent behaviour? We have customers using Chatwoot with Microsoft Email in production, haven't heard a complaint recently.

Anyway, happy to help with the implementation if required.

@pranavrajs pranavrajs added the feature-request This label is used to track new features which is a significant change to existing product label Mar 23, 2023
@evolve2k
Copy link
Author

evolve2k commented Mar 23, 2023

I would guess that they setup in 2022 before this came into affect, it probably only affects new people setting up in 2023.

If they registered this year, they likley registered a multi-tenancy access app had to get their organisation verified, which is a more involved process and requires providing legal documentation, accepting a bunch of additional terms and manual approvals on Microsofts end. It's much much better if we can avoid verification.

This PR means for single tenancy access organisational verification is avoided and Microsofts new required approach for single tenacy access is supported.

The reference PR is working and we're using it internally.
The help we need is to add tests so that it's in a state to merge.

@sojan-official sojan-official changed the title Getting Microsoft Email working with Chatwoot in 2023 [CW-1400] Getting Microsoft Email working with Chatwoot in 2023 Mar 23, 2023
@sojan-official sojan-official added the investigation This issues needs more work before it can be picked up for development label Mar 28, 2023
@evolve2k
Copy link
Author

evolve2k commented Apr 3, 2023

Checking in on this one.

@sojan-official sojan-official added this to the March 28 - April 4 milestone Apr 3, 2023
@sojan-official sojan-official modified the milestones: March 28 - April 4, 🧉 April 11 - April 17 Apr 10, 2023
@sojan-official sojan-official modified the milestones: 🧉 April 11 - April 17, 🧞‍♂️ April 18 - April 25, 🧛‍♂️ April 11 - April 19 Apr 17, 2023
@sojan-official sojan-official modified the milestones: 🧛‍♂️ April 11 - April 19, 🪴 May 2 - May 8, 🚄 May 9 - May 15 May 1, 2023
@sojan-official sojan-official modified the milestones: 🚄 May 9 - May 15, 🧺 May 16 - May 22 May 15, 2023
@tejaswinichile
Copy link
Contributor

  1. The specs which were passing not passing after rails 7 upgrade.
  2. Mainly the issue is with action-mailer base delivery method.

@sojan-official sojan-official modified the milestones: 🧺 May 16 - May 22, 🎎 May 23 - May 29 May 22, 2023
@grupale
Copy link

grupale commented Jun 8, 2023

Hi @sojan-official , @tejaswinichile any update about this issue? Thanks

@uro
Copy link

uro commented Sep 20, 2023

Hey Folks, What's the status of this issue? Is it available for early tests anywhere?

@evolve2k
Copy link
Author

evolve2k commented Sep 20, 2023

I just looked at #6878 and it was all merged 11 April 2023.

So this should all be in place and working now as far as I can tell.

If you are having a similar issue, I’d suggest to raise a new Issue.

@sojan-official sojan-official removed this from the 📮 May 30 - June 5 milestone Feb 1, 2024
@sojan-official sojan-official closed this as not planned Won't fix, can't repro, duplicate, stale Feb 1, 2024
Copy link

github-actions bot commented Mar 2, 2024

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request This label is used to track new features which is a significant change to existing product investigation This issues needs more work before it can be picked up for development linear
Projects
None yet
Development

No branches or pull requests

6 participants