merge#2
Open
Yshayy wants to merge 447 commits into
Open
Conversation
Add Actor UUID to push and branch create events for Bitbucket
* Add support for Github release webhook
* (feat) initial commit for harness code * (feat) harness, add update delete create file
(feat) harness, add repo list
(feat) harness, add webhook parsing
fetch branch for bitbucket onprem
(feat) harness, add list commits / branches
(feat) harness, add user and compare branches
(feat) harness, fix create branch, PR calls
- Replace /2.0/workspaces with /2.0/user/workspaces for org listing - Replace /2.0/repositories with workspace-scoped aggregation for repo listing - Replace /2.0/user/permissions/repositories with workspace-scoped endpoint - Add workspace extraction from client URL with fallback iteration
…ry listing - Enhanced the List and ListV2 methods to support pagination across multiple workspaces. - Introduced a new method, fetchReposWithPagination, to handle repository fetching with pagination logic. - Updated tests to validate pagination behavior and workspace handling. - Ensured consistent ordering of workspaces by slug for deterministic pagination.
- Renamed TestList_WorkspaceSorting to TestList_WorkspaceOrder for clarity. - Removed unnecessary sort parameter verification in the test. - Updated fetchAllWorkspaces to clarify sorting behavior in comments. - Enhanced fetchReposWithPagination to better determine if more repositories exist across workspaces.
- Enhanced FindPerms method to first attempt fetching permissions using a provided repository identifier. - Introduced findPermsWithIdentifier to streamline permission retrieval based on workspace extraction. - Updated findPermsAcrossWorkspaces to ensure accurate error handling and repository not found messages. - Refactored fetchReposWithPagination to utilize a paginationState struct for better state management during repository fetching across workspaces.
…edge cases - Introduced new test cases for repository permission fetching, covering scenarios such as non-existent repositories, workspace fetch failures, and empty permissions. - Added tests for repository listing, including cases with no workspaces, large page sizes, and exact page boundaries. - Enhanced error handling and validation in the repository service methods to ensure robustness against various edge cases.
…fetching - Updated List and ListV2 methods to allow fetching repositories directly from a specified URL. - Implemented pagination handling for the new URL-based fetching mechanism. - Ensured consistent error handling and response management for improved robustness.
…and consistency - Renamed methods for fetching repositories and calculating slice indices to improve clarity and maintainability. - Updated method signatures to reflect new naming conventions, ensuring consistency across the codebase. - Enhanced comments to provide better context for method functionality.
…n logic - Renamed test methods to improve clarity and consistency in naming conventions. - Updated test cases to reflect changes in method names related to slice index calculations. - Enhanced comments for better understanding of test functionality and expected outcomes.
feat: [PIPE-32185]: handle deprecated cross-workspace API endpoints
* 3a1cf9 include subgroups for listing gitlab projects
scm/driver/bitbucket: fix multiple dropped errors
… silently swallowing them (#100002) * 10a411 refactor: Clean up error handling and remove unused tests in Bitbucket driver * b1a704 fix: [PIPE-32774]: Removed extra comments * 0d256b fix: [PIPE-32774]: Add UT's * ec9df7 fix: [PIPE-32774]: Surface Bitbucket 429 rate-limit errors instead of silently swallowing them
…point fix: use non-admin endpoint for repository permission checks
… PR webhooks (#100003) * e15e87 fix: revert to time.Unix(ms/1000) — keep call sites unchanged * 32d51a fix: [PIPE-34150]: Use time.UnixMilli, add comments, refresh goldens * dfaef0 fix: [PIPE-34150]: Correct ISO 8601 layout list in epochOrISO unmarshaler * 906822 chore: remove raw payload reference files from testdata * 87c270 test: [PIPE-34150]: Add real-world FNB Bitbucket DC 10.3 push update payload test * 3ac201 test: [PIPE-34150]: Add real-world FNB Bitbucket DC 10.3 push payload test * e5a0b4 fix: [PIPE-34150]: Handle ISO 8601 date strings in Bitbucket DC stash PR webhooks
added url split strategies to be used by gitlab
* 9ec553 Merge branch 'master' of https://git0.harness.io/l7B_kbSEQD2wjrM7PShm5w/PROD/Harness_Commons/go-scm into feat/STO-11679/pr-comment-api-update * 4c15b9 feat:[STO-11679]: cleanup * bb75ba feat:[STO-11679]:Added create comment API changes * d87d12 feat:[STO-11679]:Added create comment API changes
…tus (#358) * feat: [CODE-5421]: add normalized CheckHook + GitHub check_run/status parsing Introduce scm.CheckHook, a provider-agnostic CI-check webhook carrying a batch of scm.Check entries (name, status, conclusion, target URL, sha, timestamps, optional PR). The batch shape accommodates providers that report multiple checks per event (e.g. GitLab pipeline jobs) while GitHub/Bitbucket populate a single entry. GitHub driver now parses check_run and status events (previously ErrUnknownEvent) into CheckHook. Status carries the normalized ExecutionStatus while Conclusion preserves the raw provider outcome so richer GitHub conclusions (neutral, timed_out, ...) are not lost. Adds HookEvents.Check for subscription and fixture/golden tests for both events. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * feat: [CODE-5421]: subscribe GitHub check_run/status via HookEvents.Check Wire the new HookEvents.Check flag through convertFromHookEvents so CreateHook registers both check_run and status GitHub events when a caller subscribes to checks. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
…ntents API (#100006) * 873441 fix: [CDS-125782]: Use int64 for Size field in parity with proto3 * 7c5295 test: [CDS-125782]: Add tests for large file and empty file content responses * facf81 feat: [CDS-125782]: Propagate size and encoding fields from GitHub Contents API
BitBucket closing the PR
- Add workspace and repo fields to wrapper struct - Add SetWorkspace() function to configure workspace - Use workspace when available to avoid account scope requirement - Fix fetchReposWithPagination to check for configured workspace - Update test files to use new wrapper struct format
This reverts commit 66cedce.
…arness Git providers (#100007) * 07f1c1 feat: [IAC-7349]: Support comment list and delete for BitBucket and Harness Git providers
… prefix-based branch search (#100008) * aaa5f2 added a comment that pagination is not supported * 651bf4 removed log line * 991ded added log line * e92876 added log line * 1dd54a added log line * 7e541f added search term support for listing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.