unkn0w / noobs

Kolekcja skryptów do szybkiego stawiania usług na serwerach Mikrusa
MIT License
251 stars 64 forks source link

fix #190 chce_ssh_config.sh - uzupełnienie danych #194

Closed stefopl closed 11 months ago

stefopl commented 1 year ago

Moja propozycja: GitHub Actions, który aktualizacje plik servery.txt w głównym katalogu noobs na podstawie danych z https://mikr.us/serwery.txt. Akcja wykonuje się po kązdym push, merge pull request i raz w tygodniu.

Sam skrypt chce_ssh_config.sh pobiera już listę serwerów z tego pliku. Może pozostałe skrypty również mógłby wykorzystywać ten plik.

Dodatkowo PR zawiera poprawkę, którą wpływa na czytelność kluczy w authorized_keys, o czym wspominałem w issue.

Plik serwery.txt nie jest załączony w PR, bo powinien się utworzyć po scaleniu PR. Tylko nigdy wcześniej nie robiłem PR z GitHub Actions wieć nie wiem, czy to zadziała, czy potrzebne są jeszcze jakieś kroki. A może PR musi być na main w moim repo?

loper commented 1 year ago

Nie widzę sensu zmieniać logiki i pobierać pliku "serwery.txt" - ta lista jest zawsze dostępna pod adresem mikrusa - użytkownik po pół roku nie pobierania repo może chcieć odpalić skrypt - powinien wtedy odwołać się do najnowszej wersji przez to pseudo "API", a nie plik pobrany dawno temu...

stefopl commented 1 year ago

@loper zmieniłem

Przy okazji takie pytanie, czy dodać opcję ssh config dla github? Dla mnie przydatne, ale nie wiem jak dla innych.

Jak to widzę (na przykładzie repo noobs): używamy ./chce_ssh_config.sh --github noobs

do ./ssh/configdopisuj się

Host noobs-github
    HostName github.com
    User git
    IdentityFile ~/.ssh/noobs-github

a na ekranie pokazuje się klucz, który trzeba sobie wkleić do Deploy keys https://github.com/unkn0w/noobs/settings/keys

po takim zabiegu, zamiast używać git@github.com:unkn0w/noobs.git używa się noobs-github:unkn0w/noobs.git

ferexio commented 11 months ago

@loper wypowiesz się proszę?

loper commented 11 months ago

@loper wypowiesz się proszę?

Wypowiedziałem się: loper commented on Sep 26