Skip to content

Conversation

@DeepDiver1975
Copy link
Member

Description

Related Issue

  • Fixes <issue_link>

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

phil-davis and others added 2 commits May 25, 2023 11:01
…sWithSensitiveParameters

Add loginInOwnCloud to Log::$methodsWithSensitiveParameters
…a resource which was shared with limited permissions
@DeepDiver1975
Copy link
Member Author

base branch 10.12 was created to have a base for this PR .... no real need to merge this into 10.12

but after release we need to merge this back into master- will do a new PR once we are there ....

Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

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

LGTM
should we bump version.php in this PR, or do it in a separate PR to the 10.12 branch?
(and @jnweiger will need to run the changelog... stuff anyway from the 10.12 branch when preparing the release)

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 6, 2023

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 0 Code Smells

72.7% 72.7% Coverage
0.0% 0.0% Duplication

@phil-davis
Copy link
Contributor

The code in this release/10.12.2 branch has been released.
It has also been applied to master - see PR #40828 for the details.

CI fails because it messes around trying to auto-merge stuff
https://drone.owncloud.com/owncloud/core/38565

+ git checkout master
Branch 'master' set up to track remote branch 'master' from 'origin'.
Already on 'master'
+ git fetch origin refs/pull/40827/head:
From https://github.com/owncloud/core
 * branch                  refs/pull/40827/head -> FETCH_HEAD
+ git merge 01ece70d7519e6fbea5b11fd5183c84c795e6862
Auto-merging version.php
CONFLICT (content): Merge conflict in version.php
Auto-merging tests/acceptance/features/bootstrap/OccContext.php
Auto-merging tests/acceptance/features/bootstrap/EncryptionContext.php
Auto-merging sonar-project.properties
CONFLICT (content): Merge conflict in sonar-project.properties
Auto-merging core/Command/Encryption/EncryptAll.php
Auto-merging changelog/10.12.1_2023-04-03/40709
CONFLICT (rename/delete): changelog/unreleased/40702 deleted in 01ece70d7519e6fbea5b11fd5183c84c795e6862 and renamed to changelog/10.12.1_2023-04-03/40702 in HEAD. Version HEAD of changelog/10.12.1_2023-04-03/40702 left in tree.
Auto-merging changelog/10.12.1_2023-04-03/40697
Auto-merging changelog/10.12.1_2023-04-03/40693
Auto-merging apps/federatedfilesharing/tests/NotificationsTest.php
Auto-merging apps/federatedfilesharing/tests/Controller/RequestHandlerTest.php
Auto-merging CHANGELOG.md
CONFLICT (content): Merge conflict in CHANGELOG.md
Auto-merging .drone.star
Automatic merge failed; fix conflicts and then commit the result.

Maybe we should force-merge this PR - that will get the 10.12 branch head to point to the same commit as the v10.12.2 tag, and that should make it easy and understandable for the next person, if there turns out to be any need for 10.12.3 !

@phil-davis phil-davis requested a review from jnweiger June 7, 2023 07:25
@pako81
Copy link

pako81 commented Jul 31, 2023

I guess this is not needed anymore, isn't it?

@phil-davis
Copy link
Contributor

I guess this is not needed anymore, isn't it?

correct - not needed. The code in this branch was also added to master by PR #40828

@phil-davis phil-davis closed this Jul 31, 2023
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.

5 participants