Skip to content

Add commands to handle the trusted servers from command line - #40796

Merged
pako81 merged 5 commits into
masterfrom
trusted_servers_cli_handling
Jun 5, 2023
Merged

Add commands to handle the trusted servers from command line#40796
pako81 merged 5 commits into
masterfrom
trusted_servers_cli_handling

Conversation

@jvillafanez

@jvillafanez jvillafanez commented May 17, 2023

Copy link
Copy Markdown
Member

Description

New commands to handle trusted servers from the command line

Related Issue

https://github.com/owncloud/enterprise/issues/5731

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

@jvillafanez
jvillafanez force-pushed the trusted_servers_cli_handling branch from 4ecdb3d to 21f675c Compare May 17, 2023 15:29
@jvillafanez
jvillafanez marked this pull request as ready for review May 17, 2023 16:15
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from update-docs Bot May 18, 2023
Comment thread apps/federation/lib/Command/TrustedServerRemove.php Outdated
Comment thread apps/federation/lib/Command/TrustedServerRemove.php Outdated
@phil-davis
phil-davis self-requested a review May 18, 2023 08:06

@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.

LGTM - I suppose that others should also review this.

Comment thread apps/federation/lib/Command/TrustedServerRemove.php
@pako81

pako81 commented May 18, 2023

Copy link
Copy Markdown

Apart from a small remark for the remove command, it looks ok to me 👍

@sonarqubecloud

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 A 17 Code Smells

95.3% 95.3% Coverage
0.0% 0.0% Duplication

@mrow4a

mrow4a commented May 26, 2023

Copy link
Copy Markdown
Contributor

@pako81

pako81 commented May 26, 2023

Copy link
Copy Markdown

merge this?

@pako81
pako81 merged commit 1624c63 into master Jun 5, 2023
@delete-merged-branch
delete-merged-branch Bot deleted the trusted_servers_cli_handling branch June 5, 2023 08:58
shokri-navid pushed a commit to pondersource/core that referenced this pull request Jul 16, 2023
…d#40796)

* Add commands to handle the trusted servers from command line

* Add changelog entry

* Add unit tests

* Improve messages

* Add additional message
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants