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

Nekonzistentní chování faset #194

Closed
zabak opened this issue Oct 20, 2022 · 7 comments
Closed

Nekonzistentní chování faset #194

zabak opened this issue Oct 20, 2022 · 7 comments

Comments

@zabak
Copy link
Collaborator

zabak commented Oct 20, 2022

Pokud vyberu Československý sport tak se mi automaticky zobrazí ve fasetě rozsah roků 1953 - 1989. Pokud přepíšu rok 1953 na 1966 a dám enter, tak se v aktivních filtrech zobrazí 1953 - 1989 a poté co se natáhnou data ze serveru, tak se aktivní filtr změní na 1966 - 1989. To je dívné chování, mělo by se rovnou zobrazit to co bylo zadáno, tj. 1966 - 1989.

Když pak přejdu na hlavní stránku a znovu zvolím Československý sport, tak se mi zobrazí ve fasetě to 1966 - 1989 a odpovídající čísla (ale aktivní filtr není), ale na pozadí se spustí dotaz bez filtrů a po chvíli se faseta změní na 1953-1989 a zobrazí se všechno.

Tohle chování je obecné, platí pro všechny fasety. Systém by si měl buď pamatovat aktivní filtry nastavené pro jednotlivé metatituly a když se na daný. metatitul v rámci jedné session vrátím, tak ty filtry zachová, nebo by měl rovnou při novém příchodu na metatitul nebo jiné změně faset zobrazit až ty správné výsledky, ne ty předchozí nakešované.

jahhoo pushed a commit that referenced this issue Oct 25, 2022
jahhoo pushed a commit that referenced this issue Oct 25, 2022
@Kretiss
Copy link
Collaborator

Kretiss commented Oct 25, 2022

@zabak Mělo by to být vše opraveno, upravil jsem filtry, ale by se ně nedalo klikat pokud se načítají data a opravil část kódu, co obstarávala ty requesty. Bylo tam dost chyb, ten request běžel na pozadí i pokud se přešlo na jinačí část aplikace a blokoval ostatní requesty, které byly ve frontě.

Ve Firefoxu to nefunguje korektně, na to jsem založil issue u použité knihovny

K totomu bych měl dotaz
Máte v plánu produkci nasazovat pod nějakou doménou? Bylo by vhodné tam mít SSL certifikát (teď si komunikaci může odchytnout kdo chce) + HTTP/2 (více vláken pro requesty -> HTTP/1.1 jede sekvenčně). HTTP/2 je vázané na HTTPS, takže to bez SSL certifikátu nepojede

jahhoo pushed a commit that referenced this issue Oct 25, 2022
jahhoo pushed a commit that referenced this issue Oct 25, 2022
@zabak
Copy link
Collaborator Author

zabak commented Oct 25, 2022

@Kretiss nerozumím otázce. produkce jede na https a http/2

@Kretiss
Copy link
Collaborator

Kretiss commented Oct 25, 2022

@zabak Jo já jsem si neuvědomil, že vy tam chodíte přes doménu.. my jsme totiž dostali IP adresu. V tom případě by bylo dobré tohle blokovat, aby se tam dalo dostat jenom skrze doménu

@zabak
Copy link
Collaborator Author

zabak commented Oct 28, 2022

Požadavek na blokaci přístupu přes IP adresu - viz #197
Issue samotné je možné nasadit na produkci

@smallcubes
Copy link

@Kretiss přes jakou IP adresu na Permonika chodíte? Máte na mysli interní 10.x.x.x nebo veřejnou 195.x.x.x?

@Kretiss
Copy link
Collaborator

Kretiss commented Oct 31, 2022

@smallcubes Mám namysli veřejnou

@zabak
Copy link
Collaborator Author

zabak commented Dec 5, 2022

funguje

@zabak zabak closed this as completed Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants