Improve logs download: adding query parameters

[tom1145]

Fixes #320.

Changes proposed in this PR:

• adds filter inputs for downloading the logs

[jamiehewitt15]

Can you pull the latest changes from origin develop to fix the issue with the test?

[tom1145]

Can you pull the latest changes from origin develop to fix the issue with the test?

I made a sync on both of them now. Thanks!

[jamiehewitt15]

I made some changes to make sure this works with the new authentication that's required to download the logs + updated the styling. Now using MUI as a simple way to improve the UX.

[jamiehewitt15]

@bogdanfazakas @paulo-ocean @mariacarmina Can you have a look at this PR when you have time? Thank you

[paulo-ocean]

lgtm

[bogdanfazakas]

Hey @jamiehewitt15 is download logs functionality working on your end? i always end up with a 403 with invalid signature error.

[jamiehewitt15]

Thanks for testing this @bogdanfazakas. The expiry timestamp in your request is 1713301201844 = Tuesday, 16 April 2024 so it's already expired. It should ask you for another signature in this situation, so I'll look into why that's not happening.

[jamiehewitt15]

@bogdanfazakas I've introduced some improvements that should solve your issue:

1. signature and expiry timestamp are only stored in local storage if they are still valid
2. they are only used, when retrieved from local storage if they are still valid
3. if the signature is from a different account to the one currently connected, they are removed.
4. Better error responses on the auth function.

Could you test it again from the same browser? It should recognise that your expiryTimestamp is in the past and show you the unlock button so that you sign another message.

[bogdanfazakas]

@bogdanfazakas I've introduced some improvements that should solve your issue:

1. signature and expiry timestamp are only stored in local storage if they are still valid
2. they are only used, when retrieved from local storage if they are still valid
3. if the signature is from a different account to the one currently connected, they are removed.
4. Better error responses on the auth function.

Could you test it again from the same browser? It should recognise that your expiryTimestamp is in the past and show you the unlock button so that you sign another message.

Nice fix, immediately asked me to unlock the admin features with that came the sign again operation, then download logs worked fine. 👍

[jamiehewitt15]

Nice fix, immediately asked me to unlock the admin features with that came the sign again operation, then download logs worked fine. 👍

Excellent, thanks for testing