Skip to content

chore(ci): Bump starter workflow to v5 and restore Microsoft.Graph reference#35

Merged
spbsoluble merged 5 commits into
release-4.2from
logging-fix
Jun 25, 2026
Merged

chore(ci): Bump starter workflow to v5 and restore Microsoft.Graph reference#35
spbsoluble merged 5 commits into
release-4.2from
logging-fix

Conversation

@spbsoluble

@spbsoluble spbsoluble commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bumped starter workflow from v4 to v5
  • Restored Microsoft.Graph package reference that was incorrectly removed in a prior commit — it is actively used throughout GraphClient.cs and cannot be satisfied transitively

Test plan

  • CI build passes

@spbsoluble spbsoluble changed the title Logging fix chore(ci): Bump starter workflow to v5 and restore Microsoft.Graph reference Jun 18, 2026
@spbsoluble spbsoluble requested a review from rcpokorny June 18, 2026 20:54
@spbsoluble spbsoluble merged commit 7b0fd5b into release-4.2 Jun 25, 2026
@spbsoluble spbsoluble deleted the logging-fix branch June 25, 2026 19:44
spbsoluble added a commit that referenced this pull request Jun 25, 2026
* 82249 add pam support (#31)

* - Updated the following packages to eliminiate vulnerabilities:
    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

* Update generated docs

* Misspelled words.

* Added PAM support

* Added implementation of PAM to all jobs

* updating unit tests for PAM Tests

* Updated test for PAM Provider

* Fixed PAM Issue

* Update generated docs

* Modified logging so not to show sensitive data.

* Changed logging message from 'Application ID' to 'Object ID'.
Removed additional logging of sensitive information.

* #85686 - Removed all mentions of UseSSL Flag

* Update generated docs

* Removed SSL Info

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

* Logging fix (#33)

* chore: update dependencies 

Updates:

    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* [slip ci] Merge 4.1.2 to main (#28)

* chore: update dependencies 

Updates:

    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* Fix: Update packages from previously revoked signature versions

* Updated Keyfactor packages to resolve an issue with a revoked certificated.

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

---------

Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* [skip ci] Add .NET10 support

* chore: update dependencies 

Updates:

    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* Fix: Update packages from previously revoked signature versions

* Updated Keyfactor packages to resolve an issue with a revoked certificated.

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

* Updated Frameworks to include .Net10

* Updated Frameworks to include .Net10

* Update generated docs

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

---------

Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* 82249 add pam support (#31) (#32)

* - Updated the following packages to eliminiate vulnerabilities:
    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

* Update generated docs

* Misspelled words.

* Added PAM support

* Added implementation of PAM to all jobs

* updating unit tests for PAM Tests

* Updated test for PAM Provider

* Fixed PAM Issue

* Update generated docs

* Modified logging so not to show sensitive data.

* Changed logging message from 'Application ID' to 'Object ID'.
Removed additional logging of sensitive information.

* #85686 - Removed all mentions of UseSSL Flag

* Update generated docs

* Removed SSL Info

---------

Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

* fix:Added extra logging to GraphClient

* Update generated docs

* chore: update logging of base64 contents

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>

* fix: resolve correct thumbprint when uploading application certificate

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>

* fix: logging of the certificate thumbprint

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>

* chore: update logging

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>

* Verified .Net10 was included and removed NuGet packages with vulnerabilities.

---------

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>
Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>
Co-authored-by: Morgan Gangwere <470584+indrora@users.noreply.github.com>
Co-authored-by: Matthew H. Irby <matt.irby@keyfactor.com>

* chore(ci): Bump starter workflow to v5 and restore Microsoft.Graph reference (#35)

* docs: auto-generate README and documentation [skip ci]

* docs: auto-generate README and documentation [skip ci]

---------

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>
Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: Adam Guzzo <adamguzzo@cox.net>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>
Co-authored-by: Matthew H. Irby <matt.irby@keyfactor.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
indrora added a commit that referenced this pull request Jun 25, 2026
* 82249 add pam support (#31)

* - Updated the following packages to eliminiate vulnerabilities:
    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

* Update generated docs

* Misspelled words.

* Added PAM support

* Added implementation of PAM to all jobs

* updating unit tests for PAM Tests

* Updated test for PAM Provider

* Fixed PAM Issue

* Update generated docs

* Modified logging so not to show sensitive data.

* Changed logging message from 'Application ID' to 'Object ID'.
Removed additional logging of sensitive information.

* #85686 - Removed all mentions of UseSSL Flag

* Update generated docs

* Removed SSL Info

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

* Logging fix (#33)

* chore: update dependencies 

Updates:

    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* [slip ci] Merge 4.1.2 to main (#28)

* chore: update dependencies 

Updates:

    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* Fix: Update packages from previously revoked signature versions

* Updated Keyfactor packages to resolve an issue with a revoked certificated.

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

---------

Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* [skip ci] Add .NET10 support

* chore: update dependencies 

Updates:

    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* Fix: Update packages from previously revoked signature versions

* Updated Keyfactor packages to resolve an issue with a revoked certificated.

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

* Updated Frameworks to include .Net10

* Updated Frameworks to include .Net10

* Update generated docs

---------

Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

---------

Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

* 82249 add pam support (#31) (#32)

* - Updated the following packages to eliminiate vulnerabilities:
    - Azure.Core
    - Azure.Identity
    - Newtonsoft.Json

* Update generated docs

* Misspelled words.

* Added PAM support

* Added implementation of PAM to all jobs

* updating unit tests for PAM Tests

* Updated test for PAM Provider

* Fixed PAM Issue

* Update generated docs

* Modified logging so not to show sensitive data.

* Changed logging message from 'Application ID' to 'Object ID'.
Removed additional logging of sensitive information.

* #85686 - Removed all mentions of UseSSL Flag

* Update generated docs

* Removed SSL Info

---------

Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>

* fix:Added extra logging to GraphClient

* Update generated docs

* chore: update logging of base64 contents

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>

* fix: resolve correct thumbprint when uploading application certificate

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>

* fix: logging of the certificate thumbprint

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>

* chore: update logging

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>

* Verified .Net10 was included and removed NuGet packages with vulnerabilities.

---------

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>
Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>
Co-authored-by: Morgan Gangwere <470584+indrora@users.noreply.github.com>
Co-authored-by: Matthew H. Irby <matt.irby@keyfactor.com>

* chore(ci): Bump starter workflow to v5 and restore Microsoft.Graph reference (#35)

* docs: auto-generate README and documentation [skip ci]

* docs: auto-generate README and documentation [skip ci]

* fix: force 4.2.2 release for workflows (#36)

---------

Signed-off-by: Matthew H. Irby <matt.irby@keyfactor.com>
Co-authored-by: Bob Pokorny <55611381+rcpokorny@users.noreply.github.com>
Co-authored-by: Bob Pokorny <bpokorny@keyfactor.com>
Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
Co-authored-by: Adam Guzzo <adamguzzo@cox.net>
Co-authored-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>
Co-authored-by: Matthew H. Irby <matt.irby@keyfactor.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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