Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/mirror #1683

Merged
merged 69 commits into from Jun 4, 2021
Merged

Feature/mirror #1683

merged 69 commits into from Jun 4, 2021

Conversation

eheimbuch
Copy link
Member

Proposed changes

Add mirror command and extension points.

Your checklist for this pull request

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

Contributor:

  • PR is well described and the description can be used as a commit message on squash
  • Related issues linked to PR if existing and labels set
  • Target branch is not master (in most cases develop should bet the target of choice)
  • Code does not conflict with target branch
  • New code is covered with unit tests
  • New ui components are tested inside the storybook (module ui-components only)
  • Changelog entry file created in gradle/changelog or CHANGELOG.md is updated for plugins
  • Documentation updated (only necessary for new features or changed behaviour)

Reviewer:

  • The clean code principles are respected (CleanCode)
  • All new code/logic is implemented on the right spot / "Should this be done here?"
  • UI changes fits into the layout
  • The UI views / components are responsive (mobile views)
  • Correct translations are available

Checklist for branch merge request (not required for forks)

pfeuffer and others added 30 commits June 2, 2021 10:07
@pfeuffer @sdorra
Add mirror API
3da2a29
@pfeuffer @sdorra
Add initial git implementation for mirror
950e32a
@pfeuffer @sdorra
Use logic from pull command
2a9e4c4
@pfeuffer @sdorra
Use lists for log
7877925
@pfeuffer @sdorra
Extract common logic for fetch
fe49cac
@pfeuffer @sdorra
Extract
241fe6d
@pfeuffer @sdorra
Move logic
62979bc
@pfeuffer @sdorra
Test credentials
13f7482
@pfeuffer @sdorra
Add command to service api
b0cbdb9
@pfeuffer @sdorra
Add command validation
c0ffe8f
@pfeuffer @sdorra
Add repository link provider
e95d50a
@pfeuffer @sdorra
Add test helper for rest
d22ef08
@pfeuffer @sdorra
Avoid null in mirror command request
34069dc
@pfeuffer @sdorra
Add setter for all credentials
a749d62
@pfeuffer @sdorra
Use collection for credentials
a82231a
@sdorra @pfeuffer
@eheimbuch @Pilopa
ReadOnlyCheck ExtensionPoint (#1662)
61ee87b 
The ReadOnlyCheck extension point can be used by plugins to protect a repository from write access.

Co-authored-by: René Pfeuffer <rene.pfeuffer@cloudogu.com>
Co-authored-by: Eduard Heimbuch <eduard.heimbuch@cloudogu.com>
Co-authored-by: Konstantin Schaper <konstantin.schaper@cloudogu.com>
@pfeuffer @sdorra
Add http mock helpers for json
4fd62e4
@sdorra @pfeuffer
@eheimbuch @Pilopa
Repository flag extension point (#1663)
2d34af0 
This change allows plugin authors to decorate repository cards with flags. The change modifies also how the existing flags (archive, exporting, health check failure) are rendered.

Co-authored-by: René Pfeuffer <rene.pfeuffer@cloudogu.com>
Co-authored-by: Eduard Heimbuch <eduard.heimbuch@cloudogu.com>
Co-authored-by: Konstantin Schaper <konstantin.schaper@cloudogu.com>
@pfeuffer @sdorra
Add POC for SVN mirror command
e406421
@eheimbuch @sdorra
Cleanup / small improvements
a5dac99
@eheimbuch @sdorra
Add changelog entry
229fe78
@pfeuffer @sdorra
Use byte array for certificate instead of file
e2b7874
@eheimbuch @sdorra
WIP
fc35878
@Pilopa @sdorra
fix translation issue & expose react-query for usage in plugins
24d74a2
@pfeuffer @sdorra
Use supplier to avoid circular calls
1020dc1
@pfeuffer @sdorra
Show tags from extension points in details pages
6675f3d
@eheimbuch @sdorra
Fix cache invalidation for repository import/mirror
fa320e2
@pfeuffer @sdorra
Enhance read only check interface with verb
be5e568
@Pilopa @sdorra
wip new file input
cef6e32
@pfeuffer @sdorra
Fix initial sync
e09b098
pfeuffer and others added 27 commits June 2, 2021 10:14
@pfeuffer @sdorra
Cleanup
b71f19f
@pfeuffer @sdorra
Fix build breaker
736cf48
@pfeuffer @sdorra
Revert "Fix build breaker"
db21fca 
This reverts commit 7ce306d.
@pfeuffer @sdorra
Filter mirror refs in get protocol
36a1121
@pfeuffer @sdorra
Cleanup rejected new refs correctly
0aa2fbc
@sdorra
Extend gpg api to a public key parser
6033908
@sdorra
Extend mirror command to additional public keys for changeset verific…
7cdda64 
…ation
@eheimbuch @sdorra
Update storyshots
ceb32b6
@eheimbuch @sdorra
Fix mirror unit test
7980a01
@sdorra
Add EvictingQueue
4c99245
@sdorra
Add cursor pointer to Tag if it is clickable
9be45aa
@sdorra
Allow EvictingQueue with size 0
0cbf294
@sdorra
Add basic duration component
d4c92c4
@pfeuffer @sdorra
Introduce result types instead of boolean result
f913d7a
@pfeuffer @sdorra
Log only relevant headers
78ab549
@pfeuffer
Add reasons to rejects
2e4e5be
@pfeuffer
Test to reproduce npe with annotated tags
e12e543
@sdorra
Fix tag computation for annotated tags
bf58335
@pfeuffer
Add svn exception message to log
58a271b
@pfeuffer
Fix deletion of tags in mirror ref space
c18a1d4
@pfeuffer
Add error handling
8ef1157
@pfeuffer
Introduce inner class to avoid parameter overload
b6de349
@pfeuffer
Clean up deletion
c9753c5
@pfeuffer
Add log message for update without change
0e8cd43
@pfeuffer
Enhance update filter (#1682)
e5bfcf9
@eheimbuch
Minor fixes
f63c6ba
@eheimbuch
Add changelog entry
db346d5
@eheimbuch eheimbuch merged commit dd0975b into develop Jun 4, 2021
@eheimbuch eheimbuch deleted the feature/mirror branch June 4, 2021 12:05
@sonarcloud
Copy link

sonarcloud bot commented Jun 4, 2021

Kudos, SonarCloud Quality Gate passed!

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

55.0% 55.0% Coverage
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@eheimbuch @pfeuffer @sdorra @Pilopa