navikt / helsesjekk-bot

NAV Helsesjekk slack bot
https://helsesjekk-bot.nav.no/
MIT License
4 stars 13 forks source link

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

Open eskilgh opened 2 months ago

eskilgh commented 2 months ago

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 commented 2 months ago

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

karl-run commented 2 months ago

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. :sweat_smile:

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 commented 2 months ago

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

stemmer

eskilgh commented 2 months ago

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 commented 2 months ago

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

eskilgh commented 2 months ago

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 commented 2 months ago

Huh, det er litt interessant. :sweat_smile: 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.