Merge branch 'RESTAPI-972-allow-not-bearer-JWT' into 'master'

Search for 'Bearer ' string to be removed to extract token

See merge request firecrest/firecrest!256

.gitlab-ci.yml

@@ -301,7 +301,7 @@ tag_release:
 
         else
           echo "The tag does not belong to master the branch";
-          exit 0
+          exit 1
         fi
 
   when: on_success

src/certificator/certificator.py

@@ -198,8 +198,8 @@ def check_user_auth(username,system):
 # checks JWT from Keycloak, optionally validates signature. It only receives the content of header's auth pair (not key:content)
 def check_header(header):
 
-    # header = "Bearer ey...", remove first 7 chars
-    token = header[7:]
+    # header = remove the "Bearer " string
+    token = header.replace("Bearer ","")
     decoding_result = False
     decoding_reason = ""
 
@@ -275,8 +275,8 @@ def check_header(header):
 # returns username
 def get_username(header):
 
-    # header = "Bearer ey...", remove first 7 chars
-    token = header[7:]
+    # header = remove the "Bearer " string
+    token = header.replace("Bearer ","")
     decoding_result = False
     decoding_reason = ""
 

src/common/cscs_api_common.py

@@ -100,8 +100,8 @@ def get_null_var(var):
 # checks JWT from Keycloak, optionally validates signature. It only receives the content of header's auth pair (not key:content)
 def check_header(header):
 
-    # header = "Bearer ey...", remove first 7 chars
-    token = header[7:]
+    # header = remove the "Bearer " string
+    token = header.replace("Bearer ","") 
     decoding_result = False
     decoding_reason = ""
 
@@ -179,8 +179,8 @@ def check_header(header):
 # returns username
 def get_username(header):
 
-    # header = "Bearer ey...", remove first 7 chars
-    token = header[7:]
+    # header = remove the "Bearer " string
+    token = header.replace("Bearer ","")
     decoding_result = False
     decoding_reason = ""