Skip to content

fix: log.condition with shared_secret shall not break file uploads - #41044

Merged
DeepDiver1975 merged 2 commits into
masterfrom
fix/log-condition
Feb 5, 2024
Merged

fix: log.condition with shared_secret shall not break file uploads#41044
DeepDiver1975 merged 2 commits into
masterfrom
fix/log-condition

Conversation

@DeepDiver1975

@DeepDiver1975 DeepDiver1975 commented Oct 12, 2023 •
edited by pako81
Loading

Copy link
Copy Markdown
Contributor

Description

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE
@DeepDiver1975
DeepDiver1975 force-pushed the fix/log-condition branch 2 times, most recently from 7736a43 to d56b441 Compare October 17, 2023 19:25
@sonarqubecloud

sonarqubecloud Bot commented Oct 19, 2023

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

92.6% 92.6% Coverage
0.0% 0.0% Duplication
@owncloud owncloud deleted a comment from update-docs Bot Nov 16, 2023
phil-davis
phil-davis reviewed Feb 5, 2024
View reviewed changes

@phil-davis phil-davis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This seems "a good thing", so merge it?
Comment thread changelog/unreleased/41044 Outdated
@DeepDiver1975 @phil-davis
Update changelog/unreleased/41044
759780e 
Co-authored-by: Phil Davis <phil@jankaritech.com>
@sonarqubecloud

sonarqubecloud Bot commented Feb 5, 2024

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

2 New issues
0 Security Hotspots
92.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud
@DeepDiver1975
DeepDiver1975 merged commit 19e19ae into master Feb 5, 2024
@delete-merged-branch
delete-merged-branch Bot deleted the fix/log-condition branch February 5, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - To Review

4 participants

@DeepDiver1975 @jvillafanez @phil-davis @pako81