Graph Mail 1.0.4 - #9
Merged
Klaas-Ritense merged 1 commit intoSep 9, 2026
Merged
Conversation
Reliability, security and documentation update for the Graph Mail plugin, reconciled with the fixes from #8.
Klaas-Ritense
approved these changes
Sep 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Wat dit is
Graph Mail 1.0.4. Deze fork is sinds 20 augustus doorontwikkeld en loopt inmiddels 94 commits
voor op
main; dit brengt dat werk in één keer terug, verzoend met#8.De inhoud van
#8is behouden en op twee plekken gecombineerd in plaats van overschreven:MetadataType-fix uit#8is overgenomen — die was hiernog niet bekend, en
metadata["fileName"](camelCase) leverde inderdaad stil een bijlage opdie naar het resource-UUID vernoemd was. Daar staat nu de validatie van naam en content-type
overheen die deze branch had:
#8bepaalt welke naam, deze branch of die bruikbaar is.#8(3 MiB sessie-minimum, 4 MiBschrijflimiet) is leidend; alleen de retry-methode is herschreven naar de nieuwe
wachttijdbegrenzing, zodat
addAttachmentWithRetrygeen job-executor thread meer vasthoudtvoor de volle
Retry-After.Verder zijn de
$escapedSender-fix en de save-knop-fix uit#8intact gebleven; die laatste isgecombineerd met de ambiguïteitscontrole die deze branch aan hetzelfde
subscribe-blok hadtoegevoegd.
Belangrijk voor wie upgradet
allowedSendersis verplicht. Bestaande configuraties versturen niets meer totdat delijst eenmalig is ingevuld. De whitelist wijzigen vraagt om het client secret in dezelfde
request.
tokenBaseUrl,graphBaseUrlen de timeouts zijn geen pluginproperty meer; ze staanonder
graph-mail.httpen worden bij het opstarten gevalideerd tegen een Microsoft-allowlist.Reden: het client secret wordt naar
tokenBaseUrlgePOST, dus die instelbaar houden vanuit debeheer-UI was een exfiltratiepad.
jsoupgaat van 1.17.2 naar 1.23.1 (CVE-2026-71497). De safelist hier voegt geenraw-text-elementen toe, dus deze plugin was niet exploiteerbaar — maar het is wel de
bibliotheek waar de sanitisatie op leunt.
<img src="http://...">verliest zijnsrc. Logo's ophttps://of viacid:blijvenwerken; alleen een sjabloon met een logo op
http://moet verhuizen.projectVersionstond op0.0.1terwijlplugin.properties1.0.3zei; beide staan nuop
1.0.4.graphMailPass_<activityId>op elkeprocesinstantie die mail verstuurt.
Verificatie
:backend:plugin:testktlintChecknpm run lint+ng buildDe acht acceptatiegevallen zijn handmatig doorlopen tegen een echte Entra-tenant, inclusief de
duplicaatbescherming bij een teruggedraaide transactie en een loop over dezelfde service task.