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

Dokončan projekt #2

Open
evadezelak opened this issue Jun 24, 2019 · 3 comments
Open

Dokončan projekt #2

evadezelak opened this issue Jun 24, 2019 · 3 comments

Comments

@evadezelak
Copy link
Collaborator

Pozdravljeni,
projekt sva dokončali. Bi lahko prosil preverili, če so potrebni še kakšni popravki?
Hvala in lep pozdrav

@jaanos
Copy link

jaanos commented Jun 24, 2019

Trenutno imata v auth_public.py svoje geslo - to je za vekomaj objavljeno na internetu, tako da svetujem, da ga zamenjata. Namesto tega naj bodo tam podatki za prijavo z uporabnikom javnost, ki naj pa ima zadostne pravice na tabelah - glejta pdurcik/Baza-clanstva-RDR#3. javnost ima sicer trenutno pravice CREATE in TEMPORARY na bazi sem2019_katarinab ter CREATE na shemi public - teh ne potrebuje, tako da jih odstranita (glejta zigakodric/Letalski-prevozniki#3).

Imam še par pripomb:

  • V tabeli hotel imata stolpca drzava_id in mesto_id, v tabeli mesto pa imata spet referenco na državo. Svetujem torej, da se pri hotelu države znebita, saj je tako ne potrebujeta.
  • Ko neprijavljen uporabnik pregleduje hotel, bi lahko pri obvestilu, da se mora za komentiranje prijaviti, že kar dali povezavo do prijavnega obrazca.

Dodal bom povezavo na repozitorij predmeta, tako da se lahko z @alenFMF domenita za termin zagovora. Če bosta do takrat še kaj spreminjali, mi javita, da posodobim povezavo.

@evadezelak
Copy link
Collaborator Author

Sva popravili, bi lahko prosim posodobili povezavo? Hvala

@jaanos
Copy link

jaanos commented Jul 1, 2019

Bom posodobil.

Opažam sicer, da ima uporabnik javnost še vedno pravico CREATE na shemi public, tako da lahko sam ustvarja tabele. Odstranita jo lahko tako:

REVOKE CREATE ON SCHEMA public FROM javnost;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants