Jose Celano

Results 386 comments of Jose Celano

### NOTES: - SysAdmin cannot override. If SysAdmins need to override something to avoid restarting the service they can create a ServiceAdmnin account for themselves. - All secrets should be...

Today, in our weekly meeting, I mentioned that I would move `website` section to a minimal CMS, before doing this refactoring. But in the end, we agreed to leave the...

Hi @da2ce7, since in this proposal we are going to include ALL keys in the files (unlike [mine](https://github.com/torrust/torrust-index-gui/issues/664)), I think this refactor is not necessary before adding the new settings...

> What benefit is there in this ? Is it appropriate? Should there be some form of traceability the creator? > > @josecelano It's a matter of consistency. We can...

I have reconsidered this. It migth not be a good idea after all to change the intention of the field. The Index is not a client even though it generates...

Hi @WarmBeer in the new Axum implementation I'm using an extractor: https://github.com/torrust/torrust-index-backend/blob/develop/src/web/api/v1/extractors/bearer_token.rs because some endpoints require authentication and others don't. And this is how it looks like in the upload...

Hi @mario-nt just a clarification, regarding authentication there are three types of endpoints: 1. Endpoints that don't require a logged-in user at all. 2. Endpoints that require a logged-in user...

I've been discussing with @mario-nt how to `use` the auth service in the middleware. ```rust pub async fn upload_torrent_handler( State(app_data): State, Extract(maybe_bearer_token): Extract, multipart: Multipart, ) -> Response { let...

Relates to: https://github.com/torrust/torrust-index-gui/issues/424 Hi @mario-nt. There are two functions for authorization: - `get_user_id_from_bearer_token`: this is used when a logged-in user is required - `get_optional_logged_in_user`: this is used when a logged-in...

> ### Tasks: > * [x] [User ID extractor for handlers that require an user id #445](https://github.com/torrust/torrust-index/issues/445) > * [x] [Optional user ID extractor for handlers that require an optional...