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

CDN #657

Merged
merged 3 commits into from
Oct 25, 2023
Merged

CDN #657

merged 3 commits into from
Oct 25, 2023

Conversation

tschmidtb51
Copy link
Contributor

- resolves oasis-tcs#640
- add some CDN guidance
@tschmidtb51 tschmidtb51 requested a review from sthagen October 23, 2023 13:27
@tschmidtb51 tschmidtb51 self-assigned this Oct 23, 2023
- addresses parts of oasis-tcs#640
- add CDN remark to FAQs

*Example:*

The CSAF advisory `ESA-2023-31816` was changed. Consequently, the following files in the cache need to be updated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe move the common "The CSAF advisory ESA-2023-31816 was changed." sentence in front of the two scenarios and make the "Consequently, ..." sentences different so that they do not look like a failed CPSR coding. As the first four paths are identical it took me some time to see that both examples look indeed correct.

Copy link
Contributor

@sthagen sthagen left a comment

Choose a reason for hiding this comment

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

LGTM (only nits, no problem for me to merge without additional changes)

@santosomar santosomar marked this pull request as ready for review October 25, 2023 17:10
@santosomar santosomar merged commit e4febc2 into oasis-tcs:master Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Publish article to document CSAF with CDN
3 participants