Skip to content

[BULK] [Bundle-Security] - Scheduled execution to fix known issues#1137

Merged
wadepickett merged 12 commits into
mainfrom
learn-build-service-prodbot/docutune-autopr-20260404-174206-9095421-ignore-build
Apr 11, 2026
Merged

[BULK] [Bundle-Security] - Scheduled execution to fix known issues#1137
wadepickett merged 12 commits into
mainfrom
learn-build-service-prodbot/docutune-autopr-20260404-174206-9095421-ignore-build

Conversation

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@learn-build-service-prod learn-build-service-prod Bot commented Apr 4, 2026

Remediating articles to align with content SFI guidance related to sensitive terms with GUIDs, thumbprints, and secrets.

DocuTune v1.0.0.0
CorrelationId: 6361ec82-d918-4e1c-a911-7fa74bbaad2b

#docutune

Note from reviewer to PR automation author: It would be helpful to provide a contact in these descriptions so we can communicate improvements that are needed to avoid issues that are repeated in the automation.

@learn-build-service-prod learn-build-service-prod Bot requested a review from a team as a code owner April 4, 2026 17:42
@learn-build-service-prod
Copy link
Copy Markdown
Contributor Author

#docutune-review

Comment thread iis/web-development-reference/native-code-api-reference/cglobalmodule-class.md Outdated
This should have a description of the interface instead.  Not sure why a guid of the same topic was there to begin with.  Replaced the GUID with a description as it should be.
@wadepickett
Copy link
Copy Markdown
Contributor

Some of these GUID's are actual it's the COM IID's, the interface identifier the developer literally needs to inherit from, etc. Those I am reverting, but adding clarficaiton to what they are where needeed.

However there are also some legit GUID's that need to be removed here that are GUID's for the API ref document put in a description, where instead, and actual description for the API should have been there instead. So doing that rather than putting a generic GUID in. In those cases the GUID should not have been in there in the first place.

I am correcting these at the moment...

Removed link that contained the GUID since that page no longer exists to link to.
The GIUD was part a URL, replacing with a generic GUID was not helpful.  The original topic the link was to no longer exists, so the link was removed.
The GUID replaced with a generic was part of a URL.  The topic for that URL no longer exists, so removing the link rather then creating a new broken link witha generic GUID.
Guid being replaced was part of a URL.  The original topic no longer exists, so removing the link altogether.
Removing dead link altogether, no replacement apge available.
Link needed to be updated rather than the GUID in the link being replaced with a generic GUID, which would break the URL link.
Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved after replacing the generic GUID's with either link updates or link removals. In some cases legitimate COM UUID's were referred to which developers need to know and use as is so they cannot be replaced. I noted those.

@wadepickett wadepickett merged commit d0dcd3f into main Apr 11, 2026
3 checks passed
@wadepickett wadepickett deleted the learn-build-service-prodbot/docutune-autopr-20260404-174206-9095421-ignore-build branch April 11, 2026 16:06
@wadepickett
Copy link
Copy Markdown
Contributor

I also registered the COM UUID on the SFI form to avoid having it flagged again.

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