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

Endre eksempelprogram til å bruke avsluttende skråstrek på alle relasjonsnøkler #15

Open
petterreinholdtsen opened this issue Oct 26, 2018 · 6 comments
Assignees
Labels
1/2 Estimat 0,5 dag kommentar innspill for å gjøre spesifikasjonen bedre Noark 5 v. 4 Nødvendig for ferdigstillelse av versjon 5.4

Comments

@petterreinholdtsen
Copy link
Collaborator

Dette er en kopi av mangelmelding sendt inn til Arkivverket, se også https://github.com/petterreinholdtsen/noark5-tester/blob/master/mangelmelding/sendt/2017-03-30-rel-trailing-slash.md .


       Prosjekt  NOARK 5 Tjenestegresesnitt
       Kategori  Versjon 1.0 beta
    Alvorlighet  kommentar
   Meldingstype  trenger klargjøring
Brukerreferanse  pere@hungry.com
    Dokumentdel  6.1.1
     Sidenummer  11
    Linjenummer  14
Innsendingsdato  2017-03-30

Beskrivelse

Gjelder også side 12-13 og 25-26.

I spesifikasjonen har alle navnene på HATEOAS-relasjoner avsluttende skråstrek i tabeller over relasjoner. Det samme gjelder de fleste andre plasser der http://rel.kxml.no/ er omtalt. Se f.eks. 6.1.1.3 (Opprette objekter (create)) side 16, 6.1.1.4 (Preutfylling av objekt) side 18, 7.2.1.1 (Arkiv) side 57, 7.2.1.2 (Arkivdel) side 66 og videre ut over i dokumentet. Men det er noen få eksempler der disse navnene er oppgitt uten avsluttende skråstrek, nemlig i teksten på side 12-13 og 25-26. I tillegg oppgir demonstrasjons-API-et tilgjengelig fra n5test.kxml.no konsekvent alle relasjonsnavnene uten avsluttende skråstrek. I sum blir det dermed uklart om navnene på disse relasjonsnøklene skal oppgis med eller uten avsluttende skåstrek.

Skal det være http://rel.kxml.no/noark5/v4/api/arkivstruktur/fil/ eller http://rel.kxml.no/noark5/v4/api/arkivstruktur/fil eller er det meningen at begge skal brukes om hverandre?

Det blir enklere å lage en API-klient hvis det er entydig hvordan slike relasjonsnøkler skal navngis, da en alltid vet hva en skal se etter i det som returneres fra tjeneren, og slipper å teste om en finner nøkler med eller uten skråstrek. Jeg foreslår derfor å fastslå i teksten hva som er riktig, og endre spesifikasjonen slik at alle forekomster av relasjonsnøkler oppgis på samme måte.

Jeg anbefaler å konkludere med at de fleste forekomster i spesifikasjonen er korrekt og fikse de få avvikende delene, dvs. at det alltid skal være avsluttende skråstrek i slike relasjonsnøkler.

Dette er i tråd med hvordan i hvert fall Apache håndterer katalogoppslag, der oppslag på http://tjener/katalog mottar en HTTP-omdirigering til http://tjener/katalog/ før innholdet returneres. I slike tilfeller er det altså en fordel å oppgi URL med avsluttende skråstrek, for å slippe unna med ett HTTP-oppslag i stedet for to.

Ønsket endring

Legg inn ny setning under andre setning i punkt 6.1.1 på side 11, dvs. at

«Dette gjøres med ressurslenker og relasjonslenker som inneholder
beskrivelse av ressursen med eksempler på forespørsler, resultat og
statuskoder.»

endres til

«Dette gjøres med ressurslenker og relasjonslenker som inneholder
beskrivelse av ressursen med eksempler på forespørsler, resultat og
statuskoder. Alle slike ressurslenker og relasjonslenker har
avsluttende skråstrek.»

I tillegg to to forekomstene i teksten som mangler skråstrek, dvs.
endre i del 6.1.1.1 og 6.1.1.2 på side 12 og 13:

til

samt endre i del 6.1.1.9 på side 25 og 26:

til

Respons

Ingen respons fra arkivverket så langt.

@torkjetil
Copy link

Støtter denne endringen.

petterreinholdtsen added a commit that referenced this issue Nov 5, 2018
…equirement.

Add trailing slash to all resource links without it, and mention the
need for a trailing slash where resource links are described.

This fixes issue #15.
@petterreinholdtsen
Copy link
Collaborator Author

Hm, this issue should have been closed with commit edade32.

@petterreinholdtsen
Copy link
Collaborator Author

My mistake. The change is not yet merged into master.

@petterreinholdtsen petterreinholdtsen added trenger klargjøring eksisterende tekst ikke er klar og entydig nok kommentar innspill for å gjøre spesifikasjonen bedre labels Nov 21, 2018
@hanber hanber removed the trenger klargjøring eksisterende tekst ikke er klar og entydig nok label Jan 23, 2019
@henningjensen
Copy link
Contributor

Må gå igjennom eksempelappen og sjekke rel-lenkene

@petterreinholdtsen
Copy link
Collaborator Author

Sjekket eksempelet på http://n5test.kxml.no/api/, og der returneres fortsatt uten avsluttende skråstrek:

{
  "_links":[
{"rel":"http://rel.kxml.no/noark5/v4/api/arkivstruktur",
"href":"http://n5test.kxml.no/api/arkivstruktur",
"templated":false},
{"rel":"http://rel.kxml.no/noark5/v4/api/sakarkiv",
"href":"http://n5test.kxml.no/api/sakarkiv",
"templated":false}]
}

@AnnKnu AnnKnu added the Noark 5 v. 4 Nødvendig for ferdigstillelse av versjon 5.4 label Mar 27, 2019
@AnnKnu AnnKnu added 1/2 Estimat 0,5 dag kommentar innspill for å gjøre spesifikasjonen bedre Noark 5 v. 4 Nødvendig for ferdigstillelse av versjon 5.4 and removed Noark 5 v. 4 Nødvendig for ferdigstillelse av versjon 5.4 kommentar innspill for å gjøre spesifikasjonen bedre labels Mar 27, 2019
@AnnKnu AnnKnu moved this from To do to In progress in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu moved this from In progress to To do in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu moved this from To do to UNDER ARBEID - UKE 17 - 21 timer utviklertid tilgjengelig in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu added the 6 timer Tid brukt label Mar 29, 2019
@AnnKnu AnnKnu moved this from UNDER ARBEID - UKE 17 - 21 timer utviklertid tilgjengelig to FERDIG Uke 17 in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu moved this from FERDIG Uke 17 to To do in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu removed the 6 timer Tid brukt label Mar 29, 2019
@AnnKnu AnnKnu moved this from To do to Ferdig uke 17 in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu moved this from Ferdig uke 17 to UNDER ARBEID - 21 timer utviklertid tilgjengelig uke 17 in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu moved this from UNDER ARBEID - 21 timer utviklertid tilgjengelig uke 17 to To do in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu moved this from To do to UNDER ARBEID - 21 timer utviklertid tilgjengelig uke 17 in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu moved this from UNDER ARBEID - 21 timer utviklertid tilgjengelig uke 17 to Ferdig uke 18 in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu added the 5 timer Tid brukt label Mar 29, 2019
@AnnKnu AnnKnu moved this from Ferdig uke 18 to UNDER ARBEID - 21 timer utviklertid tilgjengelig uke 17 in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@AnnKnu AnnKnu removed the 5 timer Tid brukt label Mar 29, 2019
@AnnKnu AnnKnu moved this from UNDER ARBEID - 21 timer utviklertid tilgjengelig uke 17 to To do in NOARK Tj.Gr. 5 versjon 4 Mar 29, 2019
@henningjensen henningjensen moved this from To do to UNDER ARBEID - 21 timer utviklertid tilgjengelig uke 17 in NOARK Tj.Gr. 5 versjon 4 Apr 23, 2019
@anastasiyaArk anastasiyaArk moved this from UNDER ARBEID - 21 timer utviklertid tilgjengelig uke 17 to Ferdig uke 18 in NOARK Tj.Gr. 5 versjon 4 May 2, 2019
@petterreinholdtsen petterreinholdtsen changed the title Klargjør om relasjoner skal ha avsluttende skråstrek Endre eksempelprogram til å bruke avsluttende skråstrek på alle relasjonsnøkler Jun 26, 2019
@monadani
Copy link
Collaborator

monadani commented Jul 4, 2019

Er denne nå ok?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1/2 Estimat 0,5 dag kommentar innspill for å gjøre spesifikasjonen bedre Noark 5 v. 4 Nødvendig for ferdigstillelse av versjon 5.4
Projects
No open projects
NOARK Tj.Gr. 5 versjon 4
  
Ferdig uke 18
Development

No branches or pull requests

7 participants