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

@navikt/oasis er ikke tilgjengelig på github package repo #166

Open
eskilgh opened this issue Aug 29, 2024 · 7 comments
Open

@navikt/oasis er ikke tilgjengelig på github package repo #166

eskilgh opened this issue Aug 29, 2024 · 7 comments
Labels
enhancement New feature or request

Comments

@eskilgh
Copy link
Contributor

eskilgh commented Aug 29, 2024

Hei!

Jeg vurderer å forke dette repoet til bruk hos NRK. Failer altså siden denne pakken ikke er åpent tilgjengelig. Nå er vel dette noe dere bruker som intern auth mekanisme og det må uansett byttes ut, så det er strengt tatt ikke så farlig, men tenkte jeg skulle si fra.

@MikAoJk
Copy link
Contributor

MikAoJk commented Aug 29, 2024

Er ikkje denne tilgjenngelig?
https://github.com/navikt/oasis/pkgs/npm/oasis/257288674

@karl-run
Copy link
Contributor

karl-run commented Aug 29, 2024

Hei @eskilgh!

@navikt/oasis er veldig tett knyttet til nais-plattformen, så dersom du skal gjøre et tappert forsøk på å deploye bot med dashboard i din egen org er dette en del av "NAV-baggasjen" som må ryddes opp. 😅

Det er litt dokumentert her, men det er en liten sti å gå opp. Politien har forket (https://github.com/politiet/helsesjekk-bot) og deployet den, og det er noen linjer koder som er berørt. Spesielt rundt auth.

@eskilgh
Copy link
Contributor Author

eskilgh commented Aug 29, 2024

Er ikkje denne tilgjenngelig? https://github.com/navikt/oasis/pkgs/npm/oasis/257288674

stemmer

@eskilgh
Copy link
Contributor Author

eskilgh commented Aug 29, 2024

Hei @eskilgh!

@navikt/oasis er veldig tett knyttet til nais-plattformen, så dersom du skal gjøre et tappert forsøk på å deploye bot med dashboard i din egen org er dette en del av "NAV-baggasjen" som må ryddes opp. 😅

Det er litt dokumentert (her)[https://github.com/navikt/helsesjekk-bot?tab=readme-ov-file#utenfor-nav], men det er en liten sti å gå opp. Politien har forket (https://github.com/politiet/helsesjekk-bot) og deployet den, og det er noen linjer koder som er berørt. Spesielt rundt auth.

Ja den seksjonen har hjulpet :). Foreløpig bare stubber jeg auth biten så jeg kommer videre, så må nok dette implementeres på nytt for å gi mening for oss. Tenkte uansett bare jeg skulle si fra siden jeg loka litt med å tro at jeg hadde gjort noe feil med konfigurering av PAT osv., når det rett og slett bare var at akkurat den pakken ikke er public 🧠

@karl-run
Copy link
Contributor

Aha, det er litt spicy. Så med PAT så klarer du å pulle andre @navikt pakker som @navikt/next-logger og @navikt/eslint-config-teamsykmelding? 🤔

@eskilgh
Copy link
Contributor Author

eskilgh commented Aug 29, 2024

Aha, det er litt spicy. Så med PAT så klarer du å pulle andre @navikt pakker som @navikt/next-logger og @navikt/eslint-config-teamsykmelding? 🤔

Ja det meste virker å være åpent. Jeg tror politiet sin fork ble gjort med en eldre auth pakke som ligger åpent tilgjengelig? Så derfor er det ikke noe problem å få installert pakkene der.

@karl-run
Copy link
Contributor

Huh, det er litt interessant. 😅 Ser ikke noe obvious settings på den pakken som skulle gjort den mer internt enn andre. Men det er nok "like greit". Den gjør noe runtime checks på miljøvariabler så du ville nok truffet krøll med den libben ganske kjapt. Like greit å stubbe den først som sist.

Jeg har lenge tenkt på å gjøre hele greia litt mer standalone og mindre NAV-avhengig, den må jo være helt jævlig å prodsette hos noen andre.

@MikAoJk MikAoJk added the enhancement New feature or request label Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants