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.

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

Use middleware to load org #4208

Merged
merged 9 commits into from
Oct 9, 2024

Conversation

6543
Copy link
Member

@6543 6543 commented Oct 7, 2024

close #4207

@woodpecker-bot
Copy link
Contributor

woodpecker-bot commented Oct 7, 2024

Copy link

codecov bot commented Oct 7, 2024

Codecov Report

Attention: Patch coverage is 0% with 102 lines in your changes missing coverage. Please review.

Project coverage is 26.51%. Comparing base (ddd55ee) to head (7e61bff).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
server/router/middleware/session/org.go 0.00% 46 Missing ⚠️
server/api/org_registry.go 0.00% 19 Missing ⚠️
server/api/org_secret.go 0.00% 19 Missing ⚠️
server/api/agent.go 0.00% 6 Missing ⚠️
server/api/org.go 0.00% 5 Missing ⚠️
server/router/middleware/session/user.go 0.00% 4 Missing ⚠️
server/router/api.go 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4208      +/-   ##
==========================================
+ Coverage   26.47%   26.51%   +0.04%     
==========================================
  Files         376      377       +1     
  Lines       27465    27435      -30     
==========================================
+ Hits         7270     7275       +5     
+ Misses      19530    19495      -35     
  Partials      665      665              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@anbraten anbraten changed the title Create middleware to load org and use it Use middleware to load org Oct 8, 2024
server/router/middleware/session/user.go Outdated Show resolved Hide resolved
server/router/middleware/session/org.go Outdated Show resolved Hide resolved
@6543 6543 requested a review from anbraten October 8, 2024 18:40
@6543 6543 enabled auto-merge (squash) October 9, 2024 09:51
@6543 6543 merged commit 98d7b1b into woodpecker-ci:main Oct 9, 2024
7 checks passed
@woodpecker-bot woodpecker-bot mentioned this pull request Oct 8, 2024
1 task
@6543 6543 deleted the refactor/org/middleware branch October 9, 2024 10:26
@woodpecker-bot woodpecker-bot mentioned this pull request Dec 14, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor delete or replace old code server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor: create middleware which set the org for the apis dedicated to orgs
3 participants