search

pjatk21 / pjatk-cat-guard

Bot weryfikujący użytkowników na serwerze discord przy użyciu OAuth.
https://free.itny.me
MIT License
4 stars 1 forks source link

Zdjęcia są przypisywane do złych użytkowników #12

Closed kpostekk closed 2 years ago

kpostekk commented 2 years ago

Ciężko odwzorować ten błąd, nawet nie wiadomo gdzie zacząć.

kpostekk commented 2 years ago

Usunąłem wszystkie przypisania, sprawdzę czy ponowne połączenie danych coś pomoże

rafalopilowski1 commented 2 years ago

https://github.com/pjatk21/pjatk-cat-guard/blob/e410db1f83d4132dab2714d4baace0efa12957ab/webpanel/admin.py#L297

z dokumentacji Mongo w Python user = User.objects.get(id=user_id)

rafalopilowski1 commented 2 years ago

Czy jest możliże, że kod bierze wszystkie VerificationRequest i przypisuje im dany path do photoproxy @kpostekk ?

kpostekk commented 2 years ago

Nigdzie w dokumentacji mongoengine nie znalazłem, żeby robili to tak jak ja. Zapewne to może być przyczyna.

kpostekk commented 2 years ago

Czy jest możliże, że kod bierze wszystkie VerificationRequest i przypisuje im dany path do photoproxy @kpostekk ?

Mam wątpliwości bo bug jest również widoczny w admin:admin_index gdzie oznacza wszystkich jako oczekujących na sprawdzenia zdjęcia.

kpostekk commented 2 years ago

Błąd musi myć stricte runtime'owy, ponieważ zwykły restart go naprawia i ciężko określić kiedy taki błąd powstaje.

rafalopilowski1 commented 2 years ago

Po obserwacji, chyba poprawa syntaxu pomogła - zamykam, aż znowu się nie pojawi @kpostekk

kpostekk commented 2 years ago

Błąd wciąż istnieje

rafalopilowski1 commented 2 years ago

https://github.com/pjatk21/pjatk-cat-guard/blob/e12a20cb7e7edbbb9e1373aa7efebbd739b60695/shared/documents.py#L106-L108

Może zróbmy tak:

@property
def ready(self):
    return not (not self.front.to_python() or not self.back.to_python())
kpostekk commented 2 years ago

Czemu?

rafalopilowski1 commented 2 years ago

Czemu?

self.from i self.back to jest StringField Mongo, a nie natywny str Pythona - może w tym jest problem... (nie sprawdza, czy pola wewnątrz istnieją, czy ogólna klasa istnieje)

kpostekk commented 2 years ago

Czemu?

self.from i self.back to jest StringField Mongo, a nie natywny str Pythona - może w tym jest problem...

from i back są None jeżeli nie są ustawione

kpostekk commented 2 years ago

Błąd przestał się pojawiać. Zamykam