Use CacheControlScope.Public per default

[tobias-tengler]

Fixes #7055

[github-actions]

Qodana for .NET

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

• Or via our issue tracker: https://jb.gg/qodana-issue
• Or share your feedback: https://jb.gg/qodana-discussions

[kiangkuang]

Would it be more appropriate to allow the default scope be customizable similar to DefaultMaxAge, or should that be deferred to a future enhancement?

[tobias-tengler]

Yes, this can also be done, but it wouldn't address the underlying issue. I'm going to add it later.

[github-actions]

Qodana for .NET

1 new problem were found

Inspection name	Severity	Problems
Redundant 'object.ToString()' call	🔶 Warning	1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

• Or via our issue tracker: https://jb.gg/qodana-issue
• Or share your feedback: https://jb.gg/qodana-discussions

[codecov]

Codecov Report

Attention: Patch coverage is 90.00000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 72.08%. Comparing base (723733e) to head (d0c9fb1).
Report is 3 commits behind head on main.

❗ Current head d0c9fb1 differs from pull request most recent head 9d576ad. Consider uploading reports for the commit 9d576ad to get more accurate results

Files	Patch %	Lines
...olate/Caching/src/Caching/CacheControlAttribute.cs	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7058      +/-   ##
==========================================
- Coverage   77.27%   72.08%   -5.20%     
==========================================
  Files        2632     2621      -11     
  Lines      131349   130776     -573     
==========================================
- Hits       101505    94265    -7240     
- Misses      29844    36511    +6667     

Flag	Coverage Δ
unittests	72.08% <90.00%> (-5.20%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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