Skip to content

Conversation

@gusinacio
Copy link
Contributor

@gusinacio gusinacio commented Nov 19, 2024

No description provided.

@gusinacio gusinacio changed the title refactor: remove auth check inside route refactor: use middleware for free auth token Nov 19, 2024
@gusinacio gusinacio requested a review from anirudh2 November 19, 2024 20:33
@anirudh2
Copy link
Member

I'll wait till you fix clippy!

@gusinacio gusinacio force-pushed the gustavo/tap-304-use-bearer-layer-for-static-services branch from a0c8162 to e58482a Compare November 19, 2024 20:36
@github-actions
Copy link
Contributor

github-actions bot commented Nov 19, 2024

Pull Request Test Coverage Report for Build 11921361871

Details

  • 0 of 24 (0.0%) changed or added relevant lines in 2 files are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.05%) to 71.982%

Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/service/src/routes/static_subgraph.rs 0 1 0.0%
crates/service/src/service/indexer_service.rs 0 23 0.0%
Files with Coverage Reduction New Missed Lines %
crates/tap-agent/src/agent/sender_account.rs 1 92.0%
crates/service/src/service/indexer_service.rs 2 0.0%
Totals Coverage Status
Change from base Build 11921263512: -0.05%
Covered Lines: 5498
Relevant Lines: 7638

💛 - Coveralls

@gusinacio gusinacio force-pushed the gustavo/tap-304-use-bearer-layer-for-static-services branch from e58482a to 6a29e6e Compare November 19, 2024 20:38
@gusinacio
Copy link
Contributor Author

I'll wait till you fix clippy!

done

Copy link
Member

@anirudh2 anirudh2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's cool to start to see some of the middleware stuff you wanted to incorporate. Since this took you a while to figure out, I'd recommend adding comments to anything you found tricky for other devs going forward.

@gusinacio gusinacio merged commit 0911515 into main Nov 19, 2024
10 checks passed
@gusinacio gusinacio deleted the gustavo/tap-304-use-bearer-layer-for-static-services branch November 19, 2024 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants