Skip to content

Fix hardcoding of used Graph endpoint URL#439

Open
jimdigriz wants to merge 1 commit intomguessan:masterfrom
jimdigriz:graph
Open

Fix hardcoding of used Graph endpoint URL#439
jimdigriz wants to merge 1 commit intomguessan:masterfrom
jimdigriz:graph

Conversation

@jimdigriz
Copy link

@jimdigriz jimdigriz commented Jan 26, 2026

Fix some missed plumbing.

Whilst doing so, allow the token refresh to support providing an origin header too.

@esabol
Copy link

esabol commented Jan 26, 2026

To be honest, the Origin commit should be a separate PR, imho. I also think there are more changes needed to properly add a davmail.properties parameter than what you have here, but I'm not sure. Usually new parameters are added to src/etc/davmail.properties. Might need to add it to some documentation files too.

@jimdigriz
Copy link
Author

Origin removed from this PR, it will come in another.

We already allow davmail.graphUrl but we do not actually use it.

Whilst there, also allow updating the path prefix.
@jimdigriz
Copy link
Author

Added some entries to src/etc/davmail.properties as requested for davmail.graph{Url,Prefix}.

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