jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Aliaksandra Kharushka <notificati...@github.com>
Subject Re: [jclouds/jclouds] [JCLOUDS-1428] Support for SAS token based Authentication for Azure Blob Storage (#1270)
Date Thu, 31 Jan 2019 11:57:13 GMT
ak58588 commented on this pull request.



> +   
+    /**
+    * 
+    * this method checks the length of the authentication string, and decides, which auth
method it represents. 
+    * 
+    */
+   public boolean authSAS(String credential) {
+	   int length = credential.length();
+	   
+	   boolean ruleSAS = (length < 150) && (length > 120);
+	   boolean ruleKey = (length < 100) && (length > 70 );
+	   
+	   if (ruleSAS) {
+		   return true;
+	   } else if (ruleKey) {
+		   return false;

replaced with less code 

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1270#discussion_r252638559
Mime
View raw message