Skip to content

fix(front): replace link in ssh debug dropdown #399

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
Draft
tbogosavljevic wants to merge 87 commits into
base: main
Choose a base branch
from
Draft

fix(front): replace link in ssh debug dropdown #399

tbogosavljevic wants to merge 87 commits into from

Conversation

@tbogosavljevic
Copy link
Collaborator

@tbogosavljevic tbogosavljevic commented Jun 17, 2025

📝 Description

I replaced a dead link that was left over from the documentation transition. It now leads to the correct page.

✅ Checklist

  • I have tested this change
  • This change requires documentation update

@tbogosavljevic
Copy link
Collaborator Author

tbogosavljevic commented Jun 17, 2025

Fixes #398

@tbogosavljevic tbogosavljevic changed the title fix(front): replace link in ssh debug dropdown (#398) fix(front): replace link in ssh debug dropdown Jun 17, 2025
@lucianoliberti
Copy link
Collaborator

lucianoliberti commented Sep 8, 2025

@tbogosavljevic you should set up signed commits and rebase the branch

tbogosavljevic and others added 21 commits December 10, 2025 15:15
@tbogosavljevic
fix(front): replace link in ssh debug dropdown (#398)
94178d1
@emmanuel-ferdman @lucaspin @tbogosavljevic
docs: fix RFCs guide reference (#348)
1daa423 
Signed-off-by: Emmanuel Ferdman <emmanuelferdman@gmail.com>
Co-authored-by: Lucas Pinheiro <lpinheiro@semaphore.io>
@hamir-suspect @tbogosavljevic
chore(guard): reduce guard logging volume and make logs more useful (#…
8406924 
…403)

## 📝 Description
We do not need log based metrics so the log level of "started" and
"finished" logs in the api is reduced to debug.
Also adds context metadata for grpc error logs

## ✅ Checklist
- [x] I have tested this change
- [ ] ~This change requires documentation update~
@hamir-suspect @tbogosavljevic
feat(security-toolbox): add flag for scanners (#404)
3628b4e 
## 📝 Description
Sometimes we do not want to scan licenses so this enables us to set what
scanners trivy is using

## ✅ Checklist
- [x] I have tested this change
- [ ] ~This change requires documentation update~
@TomFern @tbogosavljevic
docs: add info note about Okta usage (#401)
bf965b8 
## 📝 Description
Add info note about Okta integration to the Cloud and EE versions
Fixes: #383

## ✅ Checklist
- [X] I have tested this change
- [ ] This change requires documentation update
@dexyk @tbogosavljevic
fix(projecthub-rest-api): drop page size (#406)
71dd864 
## 📝 Description

Removed the `page_size` query parameter due to inconsistent behavior.
Pagination now relies on the default `page_size` and the `page`
parameter only.

Retained only the `x-page` and `x-has-more` response headers, which are
sufficient for paginated iteration.

Related [task](renderedtext/tasks#7953).

## ✅ Checklist
- [x] I have tested this change
- [x] ~This change requires documentation update~ - N/A
@VeljkoMaksimovic @tbogosavljevic
fix(guard): update allowed id providers (#405)
a9b6931 
## 📝 Description

In order to support saml login links without making changes to the
database, this is required
renderedtext/tasks#8141

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@forestileao @tbogosavljevic
refactor: regenerate protobuf modules (#408)
8260866
@dexyk @tbogosavljevic
feat: audit logs for "Stop Job" action (#392)
2e5bbf3 
## 📝 Description

Added auditing for stop job operation via Web and API/CLI.

Introduced audit middleware to the `public-api-gateway` service to
capture and log these actions. The middleware is designed to be
extensible for future auditing of other API operations.

Related [task](renderedtext/tasks#8123). 
## ✅ Checklist
- [x] I have tested this change
- [x] ~This change requires documentation update~ - N/A
@hamir-suspect @tbogosavljevic
fix(hooks_processor): Lower the log message level (#407)
80775ff 
## 📝 Description
Use json logging formatter when running in production env
Lower the priority of some hot path logs and introduce metrics in their
place

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@hamir-suspect @tbogosavljevic
fix(hooks_processor): explicitly add jason as production dependency (#…
56cff6e 
…410)

## 📝 Description
`jason` was not packaged into the release so json logger is crashing in
prod env and printing just a log message

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@hamir-suspect @tbogosavljevic
chore(front): Refresh protos (#412)
4153030 
## 📝 Description
Refresh protobuf definitions

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@DamjanBecirovic @hamir-suspect @tbogosavljevic
fix(hooks-processor): Don't retry processing of hooks of unsupported …
c0095f5 
…type (#413)

## 📝 Description

If the webhook received from the git provider is of an unsupported type
(e.g. merge requests on GitLab), we should not retry
processing since that is not a transient issue.

## ✅ Checklist
- [x] I have tested this change
- [x] ~This change requires documentation update~ -N/A

Co-authored-by: Amir Hasanbasic <43892661+hamir-suspect@users.noreply.github.com>
@skipi @tbogosavljevic
feat(front): partial pipeline rebuild in UI (#414)
b8801c2 
## 📝 Description
Related task: #29

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@DamjanBecirovic @tbogosavljevic
feat(v1alpha api): Allow passing parameters in Run Workflow API call (#…
d4999d8 
…323)

Changes the Run workflow API implementation to allow passing parameters
and improve performance.

Previously, the request was synchronously waiting for repo-proxy to
create a hook by retrieving data from the git provider and then
schedule the workflow on the plumber.
This had performance issues, and it did not support passing parameters to
be used in the pipeline.

The new implementation goes directly to the plumber, which only saves the
request in the database before responding, and all hook processing is
done asynchronously.
This should improve performance, and also the plumber supports passing
parameters out-of-the-box.
@VeljkoMaksimovic @tbogosavljevic
feat(rbac): enable okta login when creating integration (#416)
b522501 
## 📝 Description

renderedtext/tasks#8169

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@skipi @tbogosavljevic
toil(front): add feature flag for pipeline rebuilds in the UI (#419)
a8e58aa 
## 📝 Description
Add feature flag to control if we want to display rebuild pipeline
button in the UI

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@DamjanBecirovic @tbogosavljevic
fix(api v1alpha): Use pipeline_file as param name in Run wf API (#420)
47af2ae 
In the recent changes to how the Run workflow API works in the backend,
we changed by mistake the expected name for the parameter that holds the
name of the pipeline file. This PR fixes that issue and adds a test to
verify that the request is properly formed based on the input
parameters.
@skipi @tbogosavljevic
feat(security-toolbox): allow scanners to be configured (#421)
6f4d195 
This PR enables passing a list of scanners to be used for policy checks.
@skipi @tbogosavljevic
fix(velocity): properly handle reports exceeding size limit (#395)
ec07d8a 
## 📝 Description
This version of Sonic properly passes the `io.Reader` errors to the
decoder.

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@VeljkoMaksimovic @tbogosavljevic
toil(guard): Error handling in update emails script (#423)
b1ee30e 
## 📝 Description

While I was working on [this
task](renderedtext/tasks#8114 (comment)),
I encountered an error with the update script, when users GH API token
is no longer valid. This fixes it. Also, the script is duplicated over 2
services, which is not needed

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
radwo and others added 22 commits December 10, 2025 15:16
@radwo @tbogosavljevic
toil(front): ensure resource ownership match for test results (#478)
725916f 
## 📝 Description

N/A

## ✅ Checklist
- [x] I have tested this change
- [x] ~This change requires documentation update~
@hamir-suspect @tbogosavljevic
fix(branch_hub): Implement archive action and respond with status :OK (…
4622654 
…#479)

## 📝 Description
There is an issue with processing branch deletion webhooks from
bitbuckets and gitlab because `branch_hub` `ArchiveResponse` does not
set ok status in the response, (this expectation is set
[here](https://github.com/semaphoreio/semaphore/blob/main/hooks_processor/lib/hooks_processor/clients/branch_client.ex#L165)),
This PR resolves the response and also implements archive action.

## ✅ Checklist
- [ ] I have tested this change
- [ ] This change requires documentation update
@VeljkoMaksimovic @tbogosavljevic
Restrict cache on forked prs (#480)
c3f003b 
## 📝 Description

renderedtext/project-tasks#2647

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update: To be done
@radwo @tbogosavljevic
fix(repository_hub): URL encode branch and tag names for GitLab API (#…
bb990a3 
…481)

## 📝 Description

Branch and tag names containing forward slashes (e.g.,
"feature/branch-name", "release/v1.0.0") were causing 404 errors when
making API requests to GitLab. Added URI.encode with char_unreserved
predicate to properly encode special characters in names according to
RFC 3986.

## ✅ Checklist
- [x] I have tested this change
- [x] ~This change requires documentation update~
@VeljkoMaksimovic @tbogosavljevic
fix(front): pipeline status message shows who triggered pipeline corr…
cbdfb26 
…ectly (#483)

## 📝 Description

renderedtext/tasks#8347

## ✅ Checklist
- [x] I have tested this change
- [x] ~This change requires documentation update~
@VeljkoMaksimovic @tbogosavljevic
fix(zebra): add ff for cachehub protection (#486)
134b4b9 
## 📝 Description

renderedtext/project-tasks#2647

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@hamir-suspect @tbogosavljevic
feat: show artifact size in UI (#485)
c07117d 
## 📝 Description
Adds file size to artifacts UI

## ✅ Checklist
- [ ] I have tested this change
- [ ] This change requires documentation update
@VeljkoMaksimovic @tbogosavljevic
feat(notifications): add authorization before sending notification (#465
90bcb1b 
)

## 📝 Description

renderedtext/project-tasks#2659

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@VeljkoMaksimovic @tbogosavljevic
docs: Add note for cache in forked prs (#497)
2b2e8f8 
## 📝 Description

renderedtext/project-tasks#2647

## ✅ Checklist
- [x] ~I have tested this change~
- [x] This change requires documentation update
@hamir-suspect @tbogosavljevic
chore(github_hooks): upgrade rails version (#506)
69ca01a 
## 📝 Description
Upgrade of rails version

## ✅ Checklist
- [ ] I have tested this change
- [ ] This change requires documentation update
@VeljkoMaksimovic @tbogosavljevic
feat(front): display who was last to modify notification (#505)
d5ca558 
## 📝 Description

renderedtext/project-tasks#2659

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@VeljkoMaksimovic @tbogosavljevic
toil(front): Add note about the project access when modifying notific…
df37a16 
…ation (#508)

## 📝 Description

Small UI improvement, adding a note to the notification form
renderedtext/project-tasks#2659

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@skipi @tbogosavljevic
Fail fast strategy should properly mark ppl as failed when one of the…
aac3063 
… jobs fail (#507)
@TomFern @tbogosavljevic
docs: update to v1.4 and add latest changes (#512)
fc16a88 
## 📝 Description
Documentation change. Add e installation instructions for v1.4 on CE and
EE. Also add the latest changes:
- Notifications are only sent to users with access to the project
- Draft PR can be disabled
- There is a new Rebuild Pipeline button

Fixes #511

## ✅ Checklist
- [X] I have tested this change
- [ ] This change requires documentation update
@skipi @hamir-suspect @tbogosavljevic
Add service account feature (#466)
c271320 
## 📝 Description
Combines front/backend changes for service account functionality.

## ✅ Checklist
- [x] I have tested this change
- [x] This change requires documentation update

---------

Co-authored-by: Amir Hasanbasic <ahasanbasic@renderedtext.com>
Co-authored-by: Amir Hasanbasic <43892661+hamir-suspect@users.noreply.github.com>
@hamir-suspect @tbogosavljevic
feat(github_hooks): handle service account github repo host account (#…
4b6d92d 
…519)

## 📝 Description
Service accounts do not have `github_repo_host_account` hence we use
their semaphore credentials when injecting a workflow request (either
manual task run or manual workflow trigger via API)

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@skipi @tbogosavljevic
fix(github-hooks): support ready_for_review hook from github (#518)
e7ddc87 
## 📝 Description
Draft PRs that are converted to regular PRs don't trigger builds because
the GitHub webhook action `ready_for_review` is not in the list of
supported actions in the webhook filter.

## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@skipi @tbogosavljevic
fix(guard): differentiate between various types of GRPC errors (#522)
b4efe52
@skipi @tbogosavljevic
fix(guard): use correct field for generating service account email do…
6bffc78 
…mains (#523)
@skipi @hamir-suspect @tbogosavljevic
fix(front): remove redundant checks from add people button visibility…
a58b285 
… check (#431)

## 📝 Description

[_add_people_button](https://github.com/semaphoreio/semaphore/blob/d9a95d1890e5551a67c8071340b065c8bc61d213/front/lib/front_web/templates/people/members/_add_people_button.html.eex#L3)
partial already performs required checks. We only need to check if we're
on the `on_prem` environment.
Introduces `github_user_provider`, `gitlab_user_provider` and
`bitbucket_user_provider` feature flags so OS installations can enable
user authentication integration with git providers.
Introduces `user_repos_access` feature flag for specific setups that
need to restrict users from connecting their git provider account to
semaphore account.
## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update

---------

Co-authored-by: Amir Hasanbasic <ahasanbasic@renderedtext.com>
Co-authored-by: Amir Hasanbasic <43892661+hamir-suspect@users.noreply.github.com>
@dexyk @tbogosavljevic
fix(hooks_processor): add support for handling tag deletion hooks in …
8966404 
…Bitbucket worker (#524)

## 📝 Description
Currently, the hooks processor does not handle tag deletion webhook
events from Bitbucket.
This PR adds support for processing that event type and ensures the
corresponding branch is archived.

Fixes [the task](renderedtext/tasks#8399).

## ✅ Checklist
- [x] I have tested this change
- [x] ~This change requires documentation update~ N/A
@hamir-suspect @tbogosavljevic
fix(rbac_ee): add debug logs (#528)
c64dfa8 
## 📝 Description
<!-- Describe your changes in detail -->

## ✅ Checklist
- [ ] I have tested this change
- [ ] This change requires documentation update
@tbogosavljevic tbogosavljevic marked this pull request as draft December 10, 2025 14:18
@tbogosavljevic
Copy link
Collaborator Author

tbogosavljevic commented Dec 10, 2025

Apologies if I blew up your inbox as codeowner. I made a mistake 😵‍💫

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

Reviewers

@skipi skipi skipi approved these changes

@hamir-suspect hamir-suspect hamir-suspect approved these changes

@VeljkoMaksimovic VeljkoMaksimovic Awaiting requested review from VeljkoMaksimovic

@forestileao forestileao Awaiting requested review from forestileao

@DamjanBecirovic DamjanBecirovic Awaiting requested review from DamjanBecirovic DamjanBecirovic is a code owner

@dexyk dexyk Awaiting requested review from dexyk dexyk is a code owner

@loadez loadez Awaiting requested review from loadez loadez is a code owner

@radwo radwo Awaiting requested review from radwo radwo is a code owner

@TomFern TomFern Awaiting requested review from TomFern

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.