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

Implemented OAuth support for Exchange Online and upgraded target framework to .net 4.8 #135

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

holzerseb
Copy link

OAuth support for Exchange Online was added. Basic authentication can still be used by omitting the EWSOAuthSecret config element. For OAuth, an Exchange application with sufficient API permissions are required. The Client Secret for the Exchange application must be encrypted via DPAPI similar to the password file. Reference Config and Config Examples have been updated to and annotated with some descriptions.
Additionally, the Target Framework was bumped to .NET 4.8.0 in order to support Azure DevOps Server.

I could not verify that Basic Authentication still works, since my EWS environment doesn't allow Basic Auth anymore. Please check that the old method still works, but from the code alone nothing should have changed.
I've verified OAuth in my environment and it works like charm.

Sebastian Holzer added 8 commits June 3, 2024 12:40
added OAuth authentication support; still to be tested/reviewed
Upgraded target framework to .net 4.8.0 in order to support Azure DevOps Server;
Updated Microsoft.Exchange.WebServices nuget package from 1.2 to 2.2;
… encrypted file rather than plaintext in the config, must be generated with DPAPI;
@holzerseb
Copy link
Author

@microsoft-github-policy-service agree

@holzerseb
Copy link
Author

@davisjr Hi sorry to bother, but I saw the PR was approved and checks went through but never merged, presumably because of the agree message being sent too late. Can you merge it? Thanks

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