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

Flytting av http://rel.kxml.no/noark5/ #55

Closed
henningjensen opened this issue Nov 27, 2018 · 15 comments · Fixed by #167
Closed

Flytting av http://rel.kxml.no/noark5/ #55

henningjensen opened this issue Nov 27, 2018 · 15 comments · Fixed by #167
Assignees
Labels
5 Estimat 5 dager Noark 5 v. 4 Nødvendig for ferdigstillelse av versjon 5.4

Comments

@henningjensen
Copy link
Contributor

Løsningen bør flyttes til arkivverkets domene. I tillegg bør formen på den revurderes, da den er fryktelig tung å vedlikeholde.

@henningjensen
Copy link
Contributor Author

Oppgaven innebærer:

  • Arkivverket må opprette nytt subdomene, rel.arkivverket.no
  • Nytt subdomene dirigeres til dagens rel.kxml.no
  • Alle lenker i standarden må endres til rel.arkivvkerket.no
  • Alle lenker i eksempelapplikasjonen må endres til rel.arkivverket.no

@petterreinholdtsen
Copy link
Collaborator

petterreinholdtsen commented Jan 15, 2019 via email

@henningjensen
Copy link
Contributor Author

Det er strengt tatt KS som burde eie det domenet, da de har mest informasjon der.

Vi bytter lenke nå fordi det blir første offisielle versjon. Versjonen som hittil er utgitt er en beta versjon. Vi må kunne gjøre slike endringer selv om det vil få konsekvenser for noen systemer.

@petterreinholdtsen
Copy link
Collaborator

En må la være å gjøre slike endringer hvis spesifikasjonen skal få noen utbredelse, da ustabile standarder er lite populære å forholde seg til. Er det noen måte dette kan gjøres på en bakoverkompatibel måte? En måte er vel at alle API-tjenere i en overgangsfase tilbyr alle lenker dobbelt opp, med både gammel og ny relasjon. Da vil klienter som bruker både gammel og ny relasjon finne det de ser etter. Kan det være en løsning her? Eventuelt så kan klienter oppgi en API-versjon, og tjeneren dele ut gamle eller nye relasjoner alt etter hvilken versjon klienten oppgir å bruke.

Når det er sagt, så er det helt klart billigere å gjøre en slik endring tidlig, selv om det billigste er å ikke gjøre endringen overhodet.

@AnnKnu
Copy link
Contributor

AnnKnu commented Jan 15, 2019

Denne må avklares intern i Arkivverket. Settes på hold til den er avklart.

@hanber
Copy link
Contributor

hanber commented Jan 16, 2019

Det er vel også et spørsmål om hvor mange som i det hele tatt har tatt i bruk beta-versjonen. Hvis det er Evry og OsloMets (nikita) er det kanskje overkommelig.

@petterreinholdtsen
Copy link
Collaborator

petterreinholdtsen commented Jan 16, 2019 via email

@AnnKnu AnnKnu added the Noark 5 v. 4 Nødvendig for ferdigstillelse av versjon 5.4 label Mar 27, 2019
@AnnKnu AnnKnu added the 5 Estimat 5 dager label Mar 27, 2019
@AnnKnu
Copy link
Contributor

AnnKnu commented Mar 29, 2019

Arkivverket må opprette nytt subdomene, rel.arkivverket.no og nytt subdomene dirigeres til dagens rel.kxml.no

@henningjensen
Copy link
Contributor Author

Arkivverket må sette opp CNAME for rel.arkivverket.no som peker til rel.kxml.no

@henningjensen henningjensen assigned henningjensen and unassigned hanber and oivkru Mar 29, 2019
@petterreinholdtsen
Copy link
Collaborator

Det som skrives gir inntrykk av at planen er å bytte alle rel-lenker i spesifikasjonen. Hva er planen for bakoverkompatibilitet for klienter og tjenere som i dag bruker relasjonsnavnene som inneholder rel.kxml.no? En CNAME vil jo ikke påvirke hva klienter ser etter av relasjonsnavn, og heller hva en tjenermaskin deler ut av relasjonsnavn.

@petterreinholdtsen
Copy link
Collaborator

En ide som kanskje kan hjelpe både i denne utfordringen og i overgangen til n5v5 (#71), er å introdusere støtte for alias-relasjoner i _links, slik at en operasjon kan ha flere relasjonslenker. Det kan for eksempel formatteres slik:

{
  "_links": [
    {
      "rel": "http://rel.kxml.no/noark5/v4/api/arkivstruktur/mappe",
      "relalias": ["http://rel.arkivverket.no/noark5/v4/api/arkivstruktur/mappe"],
      "href": "somewhere"
    },
    ...
  ]
}

Etter hvert kan dette så snus til å ha de opprinnelige relasjonsnavnene som alias, og de nye som primærnavn:

{
  "_links": [
    {
      "rel": "http://rel.arkivverket.no/noark5/v4/api/arkivstruktur/mappe",
      "relalias": ["http://rel.kxml.no/noark5/v4/api/arkivstruktur/mappe"],
      "href": "somewhere"
    },
    ...
  ]
}

Foreslår her at attributten relalias peker til en lenke med alternative navn for samme operasjon/relasjon, slik at en kan håndtere flere overganger i samme API.

@petterreinholdtsen
Copy link
Collaborator

Det kan se ut til at de eneste API-klientene som er skrevet så langt er de klientene jeg har skrevet for å teste Nikita-prosjektet, samt de som brukes internt i Nikita-prosjektet. HK-data forteller at ikke har skrevet klientkode så langt, og API-et som Evry har laget følger ikke strukturen til denne spesifikasjonen og må dermed uansett skrives om til å bli i tråd med tjenestegrensesnittet. Dermed er det i realiteten kun to Nikita-prosjektet som blir direkte påvirket når relasjonsnøkkelnavnene endres.

@tsodring
Copy link
Contributor

Angående denne. nikita har oppdatert til rel.arkivverket.no

anastasiyaArk pushed a commit to anastasiyaArk/noark5-tjenestegrensesnitt-standard that referenced this issue May 9, 2019
arkivverket#37
SystemId cardinality is changed p.30


SystemID cardinality is changed, p.31

arkivverket#37
SystemID cardinality is changed, p.38

arkivverket#37
Revert "SystemID cardinality is changed, p.29"

This reverts commit 819d46a.

Revert "SystemId cardinality is changed p.30"

This reverts commit c786d78.

Revert "SystemID cardinality is changed, p.31"

This reverts commit 4b05aeb.

Revert "SystemID cardinality is changed, p.38"

This reverts commit 0dc4fda.

rel.kxml links are changed with rel.arkivverket

arkivverket#55
anastasiyaArk pushed a commit to anastasiyaArk/noark5-tjenestegrensesnitt-standard that referenced this issue May 9, 2019
@petterreinholdtsen
Copy link
Collaborator

Jeg gikk ut i fra at den nye URL-en vil bruke https, men ser at endringsforslaget fra @anastasiyaArk bruker http. Nettlesere advarer mot nettsider uten https, og bruken av ukrypterte forbindelser gjør det enklere for uhederlige aktører på nettet å bytte ut informasjon fra nettsiden under overføring. Dette og mere til gjør at jeg anbefaler på det sterkeste å bytte fra http: til https: i relasjonsnøkkelnavnene.

@henningjensen
Copy link
Contributor Author

Det er en glipp, vi skal selvsagt over på https.

henningjensen pushed a commit to anastasiyaArk/noark5-tjenestegrensesnitt-standard that referenced this issue May 21, 2019
oivkru added a commit that referenced this issue May 21, 2019
rel.kxml links are changed with rel.arkivverket
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 Estimat 5 dager Noark 5 v. 4 Nødvendig for ferdigstillelse av versjon 5.4
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants