Skip to content

Harden supply chain: message-center injection, PII in samples, workfl… - #484

Merged
soyalejolopez merged 2 commits into
microsoft:masterfrom
soyalejolopez:upstream-security-hardening
Jul 22, 2026
Merged

Harden supply chain: message-center injection, PII in samples, workfl…#484
soyalejolopez merged 2 commits into
microsoft:masterfrom
soyalejolopez:upstream-security-hardening

Conversation

@soyalejolopez

Copy link
Copy Markdown
Member

…ow perms

  • update-message-center: sanitise fetched Message Center posts against a schema whitelist and JSON.stringify + escape </script>, U+2028/2029 and & when embedding data into the generated page, preventing stored HTML/script injection from upstream feed content. Also removes the unattended 'gh pr merge --auto' step so generated changes require human review.
  • Sample data: replace real-looking tenant identities, UPNs, phone numbers and GUIDs in three sample CSVs with Contoso / 555-01xx / random-GUID placeholders (no PII).
  • build-catalog.yml: drop top-level token to 'contents: read' and grant 'contents: write' only to the publish job, so the PR-triggered validate job (which runs untrusted fork changes) can no longer write to the repo.

Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

Copilot-Session: cecb163d-04c4-4b01-ab7a-14d25f7ec690

…ow perms

- update-message-center: sanitise fetched Message Center posts against a schema
  whitelist and JSON.stringify + escape </script>, U+2028/2029 and & when embedding
  data into the generated page, preventing stored HTML/script injection from
  upstream feed content. Also removes the unattended 'gh pr merge --auto' step so
  generated changes require human review.
- Sample data: replace real-looking tenant identities, UPNs, phone numbers and GUIDs
  in three sample CSVs with Contoso / 555-01xx / random-GUID placeholders (no PII).
- build-catalog.yml: drop top-level token to 'contents: read' and grant
  'contents: write' only to the publish job, so the PR-triggered validate job (which
  runs untrusted fork changes) can no longer write to the repo.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: cecb163d-04c4-4b01-ab7a-14d25f7ec690
Empty commit to run Stage 1 PR Sweeper against the now-fixed workflow on master. Will be removed by squash-merge.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: cecb163d-04c4-4b01-ab7a-14d25f7ec690
@soyalejolopez
soyalejolopez merged commit c6fa4f4 into microsoft:master Jul 22, 2026
4 checks passed
@soyalejolopez
soyalejolopez deleted the upstream-security-hardening branch July 22, 2026 14:41
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