Merge #42: fix: [#41] add authentication bearer token to all requests

5536b1e fix: [#41] add authentication bearer token to all requests (Jose Celano)

Pull request description:

  Relates to: torrust/torrust-index-gui#412

  Add authentication bearer token to all requests.

  The Index does not have any session data on the backend. If the client does not provide the Authentication Bearer Token the request is processed as a anonymous request. Some endpoints like the one to download the torrent file return different data depending on wether the user is logged in or not. So we have to include always the token is it's available.

ACKs for top commit:
  josecelano:
    ACK 5536b1e

Tree-SHA512: bc2907486e5facfb232119c2322740897dea219506e5ff1a91b49c2da205fc79409ec063d44ca917bc8305f594fad2fcc1560a7c82dbf47275ec5d2bb970b751

Author: josecelano

package.json

@@ -1,6 +1,6 @@
 {
   "name": "torrust-index-api-lib",
-  "version": "1.0.0-alpha.4",
+  "version": "1.0.0-alpha.5",
   "description": "Contains API functions for the Torrust project.",
   "repository": "https://github.com/torrust/torrust-index-api-lib",
   "license": "SEE LICENSE IN COPYRIGHT",

src/modes/rest/resources/torrent.ts

@@ -70,9 +70,14 @@ export class TorrentResource implements IRestResource {
         this.client = client;
     }
 
+    headers(): HeadersInit | undefined {
+        return this.client.authToken ? { "Authorization": `Bearer ${this.client.authToken}` } : undefined;
+    }
+
     async getTorrentInfo(infoHash: string): Promise<TorrentResponse> {
         return await fetchGet<GetTorrentResponse>(
-            `${this.client.apiBaseUrl}/torrent/${infoHash}`
+            `${this.client.apiBaseUrl}/torrent/${infoHash}`,
+            this.headers()
         )
             .then((res) => {
                 return Promise.resolve(res.data);
@@ -84,7 +89,8 @@ export class TorrentResource implements IRestResource {
 
     async getTorrents(params: GetTorrentsParams): Promise<GetTorrentsResponseData> {
         return await fetchGet<GetTorrentsResponse>(
-            `${this.client.apiBaseUrl}/torrents?page_size=${params.pageSize}&page=${params.page - 1}&sort=${params.sorting}${ params.categories ? "&categories=" + params.categories.join(",") : ""}${ params.tags ? "&tags=" + params.tags.join(",") : ""}${params.searchQuery ? "&search=" + params.searchQuery : ""}`
+            `${this.client.apiBaseUrl}/torrents?page_size=${params.pageSize}&page=${params.page - 1}&sort=${params.sorting}${params.categories ? "&categories=" + params.categories.join(",") : ""}${params.tags ? "&tags=" + params.tags.join(",") : ""}${params.searchQuery ? "&search=" + params.searchQuery : ""}`,
+            this.headers()
         )
             .then((res) => {
                 return Promise.resolve(res.data);
@@ -98,7 +104,7 @@ export class TorrentResource implements IRestResource {
         return await fetchDelete<any, DeleteTorrentResponse>(
             `${this.client.apiBaseUrl}/torrent/${infoHash}`,
             {},
-            { "Authorization": `Bearer ${this.client.authToken}` }
+            this.headers()
         )
             .then((res) => {
                 return Promise.resolve(res.data);
@@ -112,7 +118,7 @@ export class TorrentResource implements IRestResource {
         return await fetchPut<UpdateTorrentParams, UpdateTorrentResponse>(
             `${this.client.apiBaseUrl}/torrent/${infoHash}`,
             params,
-            { "Authorization": `Bearer ${this.client.authToken}`, "Content-Type": "application/json" }
+            this.headers()
         )
             .then((res) => {
                 return Promise.resolve(res.data);
@@ -134,7 +140,7 @@ export class TorrentResource implements IRestResource {
         return await fetchPost<NewTorrentResponse>(
             `${this.client.apiBaseUrl}/torrent/upload`,
             formData,
-            { "Authorization": `Bearer ${this.client.authToken}` }
+            this.headers()
         )
             .then((res) => {
                 return Promise.resolve(res.data);
@@ -146,7 +152,8 @@ export class TorrentResource implements IRestResource {
 
     async downloadTorrent(infoHash: string): Promise<Blob> {
         return await fetchGetBlob(
-            `${this.client.apiBaseUrl}/torrent/download/${infoHash}`
+            `${this.client.apiBaseUrl}/torrent/download/${infoHash}`,
+            this.headers()
         )
             .then((blob) => {
                 return Promise.resolve(blob);
@@ -157,11 +164,9 @@ export class TorrentResource implements IRestResource {
     }
 
     async proxiedImage(url: string): Promise<Blob> {
-        const headers = this.client.authToken ? { "Authorization": `Bearer ${this.client.authToken}` } : undefined;
-
         return await fetchGetBlob(
             `${this.client.apiBaseUrl}/proxy/image/${encodeURIComponent(url)}`,
-            headers
+            this.headers()
         )
             .then((blob) => {
                 return Promise.resolve(blob);