Beheer: integreer transport security module in core #162
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Voorheen gebruikten we een referentie naar de module als verplichting. Het ADR module systeem is ontworpen zodanig dat modules optioneel zijn. Maar met deze verwijzing moet de transport security module altijd worden toegepast. Qua versiebeheer en verduidelijking van de relatie tussen ADR core en overige modules, integreren we hiermee regels van transporty security in core. Hierbij worden ook enkele regels in het format van core gegoten, zodanig dat rationale en how to test in tabel-vorm zitten. De aparte module had hiervoor aparte Markdown secties, wat niet consistent was. Als laatste zijn er wat Engels typos opgelost.
TimvdLippe
added
Scope: Klein
Deze is een restrictie van de core HTTP method regel. De core HTTP method regel specificeert tevens dat de 405 status code moet worden teruggegeven, dus dat is hier overbodig.
- Fix kapotte links - Voeg implications toe waar die eerst leeg waren - Verwijder onnodige extra sectie headers
Zodat de biblio van de organisatie en dit document correct worden samen gevoegd.
- Gebruik MAY ipv RECOMMENDED voor betere Engelse grammatica - Consolideer HTTP header filtering sectie
mrtn78
reviewed
Jan 24, 2025
mrtn78
approved these changes
Jan 24, 2025
sanderke
reviewed
Jan 24, 2025
sanderke
reviewed
Jan 24, 2025
sanderke
requested changes
Jan 27, 2025
De NCSC.TLS referentie is vervangen door de NCSC 2021 uit de organisatie configuratie. Co-authored-by: Alexander Green <alexander.green@logius.nl>
Co-authored-by: Alexander Green <alexander.green@logius.nl>
TimvdLippe
changed the title
GitHub resolvede automatisch de merge conflicts, maar gooide daarmee een deel van de wijzigingen in develop weer weg. Dat is niet de bedoeling, dus nu staan ze er weer.
Dat is de officiele locatie ipv naar datatracker.ietf.org
TheBonheurs
reviewed
Jan 29, 2025
TheBonheurs
approved these changes
Jan 31, 2025
sanderke
requested changes
Jan 31, 2025
sanderke
approved these changes
Feb 4, 2025
TimvdLippe
added
Status: Ter goedkeuring
|
Deze wijziging is gisteren 4 februari 2025 goedgekeurd in het TO. Voor gedetailleerde reviews kan er gereageerd worden op deze pull request tot uiterlijk 9 februari 2025. |
TimvdLippe
added
Status: Gereed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Voorheen gebruikten we een referentie naar de module
als verplichting. Het ADR module systeem is ontworpen
zodanig dat modules optioneel zijn. Maar met deze
verwijzing moet de transport security module altijd
worden toegepast.
Qua versiebeheer en verduidelijking van de relatie
tussen ADR core en overige modules, integreren
we hiermee regels van transporty security in core.
Hierbij worden ook enkele regels in het format van core
gegoten, zodanig dat rationale en how to test in
tabel-vorm zitten. De aparte module had hiervoor aparte
Markdown secties, wat niet consistent was.
Als laatste zijn er wat Engels typos opgelost.
Fixes Geonovum/KP-APIs#623