Skip to content

Conversation

@CDR-FarooqK
Copy link
Contributor

Checklist: (Put an x in all the boxes that apply)

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have checked that there aren't any other open Pull Requests from the same change.
  • The develop branch has been set as the base branch to merge changes of the pull request.
  • I have updated the CHANGELOG.md file as appropriate.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior? (if this is a feature change)

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:
release v1.1.1

Replaced autobuild with dotnet build.
if (configOptions.OverrideMtlsChecks.Contains(context.HttpContext.Request.Path))
{
logger.LogDebug("Overriding mtls check for {path}...", context.HttpContext.Request.Path);
_logger.LogDebug("Overriding mtls check for {path}...", context.HttpContext.Request.Path);

Check failure

Code scanning / CodeQL

Log entries created from user input

This log entry depends on a [user-provided value](1).
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 64 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@CDR-FarooqK CDR-FarooqK marked this pull request as ready for review November 27, 2023 07:28
Copy link
Contributor

@CDR-AmirM CDR-AmirM left a comment

Choose a reason for hiding this comment

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

Looks good to me

@CDR-AmirM CDR-AmirM merged commit e6cc07c into develop Nov 27, 2023
@CDR-FarooqK CDR-FarooqK deleted the releases/1.1.1 branch November 30, 2023 01:12
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.

4 participants