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

[!!!][FEATURE] Enable basic HTTP caching and use Symfony's reverse proxy #142

Merged
merged 1 commit into from
Mar 18, 2022

Conversation

eliashaeussler
Copy link
Owner

This PR adds a basic application cache layer using Symfony's reverse proxy in combination with HTTP caching methods. Since TER API requests are already cached, the cache expiry date can be used and forwarded to the client.

Related: #136

@eliashaeussler eliashaeussler added enhancement New feature or request breaking This introduces a breaking change labels Mar 17, 2022
@eliashaeussler eliashaeussler self-assigned this Mar 17, 2022
@codeclimate
Copy link

codeclimate bot commented Mar 18, 2022

Code Climate has analyzed commit 419e070 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 99.5% (0.0% change).

View more on Code Climate.

@eliashaeussler eliashaeussler temporarily deployed to dev March 18, 2022 07:16 Inactive
@codecov
Copy link

codecov bot commented Mar 18, 2022

Codecov Report

Merging #142 (419e070) into develop (31b924c) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop     #142      +/-   ##
=============================================
+ Coverage      99.44%   99.52%   +0.08%     
- Complexity        75       85      +10     
=============================================
  Files             17       18       +1     
  Lines            181      212      +31     
=============================================
+ Hits             180      211      +31     
  Misses             1        1              
Impacted Files Coverage Δ
src/Controller/AbstractBadgeController.php 100.00% <100.00%> (ø)
src/Controller/DownloadsBadgeController.php 100.00% <100.00%> (ø)
src/Controller/ExtensionBadgeController.php 100.00% <100.00%> (ø)
src/Controller/StabilityBadgeController.php 100.00% <100.00%> (ø)
src/Controller/VersionBadgeController.php 100.00% <100.00%> (ø)
src/Entity/Dto/ExtensionMetadata.php 100.00% <100.00%> (ø)
src/Service/ApiService.php 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31b924c...419e070. Read the comment docs.

@eliashaeussler eliashaeussler merged commit 5380daf into develop Mar 18, 2022
@eliashaeussler eliashaeussler deleted the feature/http-caching branch March 18, 2022 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking This introduces a breaking change enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant