search

LorenzHetterich / CTFHelperJava

2 stars 1 forks source link

Store all the Cookies #3

Closed LorenzHetterich closed 4 months ago

LorenzHetterich commented 5 months ago

Right now, the way sessions survive bot restarts is that an CTFdApi.CTFdAuth instance containing the last cookie set by /ctfd-cookie is stored. However, this does not work if multiple cookies need to be set. Instead of storing a single CTFdAuth instance, we should probably store a List of them (or cookies). But then, we should also add a command to remove a cookie or clear all cookies.

LorenzHetterich commented 4 months ago

Apparently even storing that single instance is broken. This one has low priority though (as the bot isn't restarted often and adding a cookie through /ctfd-cookie after each restart isn't that hard).

LorenzHetterich commented 4 months ago

Should be fixed with the latest rewrite.