phoseum-cli 0.0.19 → 0.0.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '092592a7b73a29fb773b199b9e29f7054da0da5701583c1162c2500a1860635f'
-  data.tar.gz: fc71817a9eb49a14d2567694efaa91ee076e991a1780b8abdbf20b3a2c2032db
+  metadata.gz: bfc9c3e5f01d6ea982796a580be0195b76156127ab1b39c3c1bfdb0d1ada4f21
+  data.tar.gz: eb9e11b414ee5be39678593668f2fa8f02477e61b1cdbd85a0d69fbdf8728d6f
 SHA512:
-  metadata.gz: 78a80cda0d947299a0b95b86f2ccebcf1df4d263bdad641ff0d0aec8d3e2445a28d2062ba52f49d520bf334051448adcf02674069992f7b55d22e1eb60e9f288
-  data.tar.gz: 2b296a8c6ab2f30de4e7c85dd0b9acf55a3c5b8130ed9887c359220da14879d2b73895894b29f450c4b4a390f04edae6473fe317874183284d336066ddaac3cb
+  metadata.gz: c433b128e7140e01364e04309bec935cf66efcfebe785ffbe7f4cefbf0dcdf007cb2f8f7f6398eb7fcae87c751fb7f4213110422b03d621120c991369277d8a6
+  data.tar.gz: 1435d44fa6d8599e5ba9492fbce3bb5e30dba57e24c225f7ed6cbfd87ae56f7f3cca5820e9c94a7d259ca652e8875ec5ea503ebb1839969561cf99c5c5f28b7c

data/bin/phoseum-cli

@@ -391,6 +391,7 @@ case options
     puts "Creating User".green if !$QUIET
     if options[:create_user]
       value=check_string_sanity(options[:create_user])
+      username_sanity(value)
       what='add_user'
       if !options[:role]
         puts "You need to select one of the valid Roles".red
@@ -401,6 +402,7 @@ case options
         print "\t"
         confirm=STDIN.getpass('Confirm: ')
         if new_password == confirm
+          password_sanity(new_password)
           user_mgmt(what,value,new_password,options[:role])
         end
       end
@@ -443,15 +445,24 @@ case options
     if options[:user]
       what='user'
       value=options[:user]
-      if confirm_action("Are you sure you want to delete the #{what} #{value}")
-        puts "Deleting #{what} #{value}".green if !$QUIET
-        if delete_user(value)
-          puts "#{value} successfully deleted.".green
+      secretinfo = api_caller({"action" => "check-secret"},true)
+      if secretinfo['error']
+        err_msg = JSON.parse(secretinfo)
+        puts err_msg['error'].red
+        puts "You cannot delete users if you don't have the Server Secret."
+        exit 1
+      end
+      if secretinfo['success']
+        if confirm_action("Are you sure you want to delete the #{what} #{value}")
+          puts "Deleting #{what} #{value}".green if !$QUIET
+          if delete_user(value)
+            puts "#{value} successfully deleted.".green
+          else
+            puts "#{value} failed to be deleted.".red
+          end
         else
-          puts "#{value} failed to be deleted.".red
+          puts "Deleting #{what} cancelled.".green
         end
-      else
-        puts "Deleting #{what} cancelled.".green
       end
     end
     if !what
@@ -460,25 +471,117 @@ case options
     end
   when -> (o) { o[:options] }
     if options[:user]
+      user_work=check_string_sanity(options[:user])
       tokeninfo = api_caller({"action" => "token-payload", "token" => "#{$config['TOKEN']}"})
       if tokeninfo['error']
-        puts "Your token seems invalid, please login again".red
+        puts "Your token seems invalid, please login again (-l or -c)".red
         exit 1
       end
+      auth_user = ''
+      role_user = ''
       if tokeninfo['success']
-        puts tokeninfo.payload
+        payload = tokeninfo['payload']
+        tkdata = JSON.parse(payload)
+        auth_user = tkdata[0]['data']['user'].clone
+        role_user = tkdata[0]['data']['role'].clone
+        puts "Current user is #{auth_user}".green
+      end
+
+      secretinfo = api_caller({"action" => "check-secret"},true)
+      if secretinfo['error']
+        err_msg = JSON.parse(secretinfo)
+        puts err_msg['error'].red
+        puts "You will only be able to change your own password."
+        if user_work != auth_user
+          puts "Requesting other user than your own at command line is wrong, aborting. (#{user_work} is not #{auth_user})".red
+          exit 1
+        end
+      end
+      if secretinfo['success']
+        puts secretinfo['success'].green
+        if role_user == "Super"
+          puts "You will be able to change any user on the system".green
+        else
+          if user_work != auth_user
+            puts "Having a regular user will now allow to change other users's options".yellow
+            puts "Requesting other user than your own at command line is wrong, aborting. (#{user_work} is not #{auth_user})".red
+            exit 1
+          end
+        end
+      end
+
+      if user_work == auth_user
+        print "Please type new password for user #{user_work} .\n\t"
+        new_password=STDIN.getpass('Password: ')
+        print "\t"
+        check_password=STDIN.getpass('Confirm: ')
+        if new_password != check_password
+          puts "Passwords won't match, try again.".red
+          exit 1
+        end
+        if change_user_password(auth_user,check_password)
+          puts "Password changed successfully".green
+          exit 0
+        else
+          puts "Could not change user password.".red
+          exit 1
+        end
+      end
+
+      if user_work != auth_user && secretinfo['success']
+        print "\n Change [P]assword, [U]sername, [R]ole from user #{user_work} :[P/U/R]: "
+        opt_action=STDIN.gets.chomp
+        if opt_action == "P"
+          print "Please type new password for user #{user_work} .\n\t"
+          new_password=STDIN.getpass('Password: ')
+          puts "testing #{new_password}"
+          print "\t"
+          check_password=STDIN.getpass('Confirm: ')
+          if new_password != check_password
+            puts "Passwords won't match, try again.".red
+            exit 1
+          end
+          password_sanity(new_password)
+          if change_user_password(options[:user],check_password,true)
+            puts "Password changed successfully".green
+            exit 0
+          else
+            puts "Could not change user password.".red
+            exit 1
+          end
+        end
+        if opt_action == "U"
+          print "\tPlease type new username for user #{user_work} : "
+          new_user_proto=STDIN.gets.chomp
+          new_user=check_string_sanity(new_user_proto)
+          username_sanity(new_user)
+          if change_username(user_work,new_user)
+            puts "Username changed successfully".green
+            exit 0
+          else
+            puts "Could not change username.".red
+            exit 1
+          end
+        end
+        if opt_action == "R"
+          print " Please select between roles [S]uper,[U]ser for user #{user_work} :[S/U]: "
+          new_role=STDIN.gets.chomp
+          if new_role == "S" || new_role == "U"
+            role_nominal = new_role == "S" ? "Super" : "User"
+            if change_role(user_work,role_nominal)
+              puts "Username changed successfully".green
+              exit 0
+            else
+              puts "Could not change username.".red
+              exit 1
+            end
+          else
+            puts "Unknown option: #{new_role} .".red
+            exit 1
+          end
+        end
       end
-      # print "Please confirm #{options[:user]} password.\n\t"
-      # current_password=STDIN.getpass('Password: ')
-      # if token = user_login(options[:user],current_password)
-      #   print "\n Change [P]assword, [U]sername, [R]ole :[P/U/R]: "
-      #   confirm=STDIN.gets.chomp
-      # else
-      #   puts "User Check Failed".red
-      #   exit 1
-      # end
     else
-    # if !what
       puts "Got no user to work with. Exiting.".red
       exit 1
     end
@@ -533,6 +636,7 @@ case options
     test
   else
     ARGV[0] = '--help'
+    local_version
     option_parser(ARGV)
     exit 1
 end

data/lib/phoseum/phoseum-cli-lib.rb

@@ -1,4 +1,12 @@
 
+# Provide some help and options!
+#
+# Example:
+#   >> options = option_parser(ARGV)
+#   => puts options[:option]
+#
+# Arguments:
+#   ARGV: (Array or command line parameters)
 def option_parser(opts)
   options = {}
   OptionParser.new do |opts|
@@ -60,7 +68,7 @@ def option_parser(opts)
       options[:role] = r
     end
 
-    opts.on("-o", "--options", "Update options from object. Use with: [user]") do |o|
+    opts.on("-o", "--options", "Update options from User. Use with: [user]") do |o|
       options[:options] = o
     end
 
@@ -88,6 +96,15 @@ def option_parser(opts)
   return options
 end
 
+# Give yes/no prompt!
+#
+# Example:
+#   >> confirm_action("message")
+#   => You must write 'YES' to confirm, otherwise NO is assumed
+#   => message :[YES/NO]:
+#
+# Arguments:
+#   msg: (String)
 def confirm_action(msg)
   puts "You must write 'YES' to confirm, otherwise NO is assumed".yellow
   print "#{msg} :[YES/NO]: "
@@ -138,16 +155,6 @@ def client_checks
     puts "I could not find a valid SERVERURL configuration. Contains: #{$config['SERVERURL']}".red
     exit 1
   end
-  if !$config['DEFAULT_SECRET']
-    puts "I could not find the DEFAULT_SECRET from Phoseum config, this will limit our actions.".red
-    exit 1
-  elsif $config['DEFAULT_SECRET'] == 'copy-secret-from-server'
-    puts "DEFAULT_SECRET from Phoseum config. Still on self generated value, copy a valid one from the server.".red
-    exit 1
-  elsif $config['DEFAULT_SECRET'] == ''
-    puts "I could not find the DEFAULT_SECRET from Phoseum config, Variable is empty.".red
-    exit 1
-  end
   if !$config['SERVERURL']
     puts "I could not find the SERVERURL from Phoseum config, this client is then useless.".red
     exit 1
@@ -228,16 +235,31 @@ def search_image(sign,album='')
   end
 end
 
-def validate_token(token)
+def api_caller(json_body,auth=false,cli=false)
   headers = {}
-  if $config['TOKEN']
-    headers={ "bearer" => "#{$config['TOKEN']}" }
-  else
-    return false
+  if !auth
+    if $config['TOKEN']
+      headers={ "bearer" => "#{$config['TOKEN']}" }
+    else
+      return false
+    end
   end
   base = URI.parse("#{$config['SERVERURL']}")
   request = Net::HTTP::Post.new(base,headers)
-  request.body = JSON.generate({"action" => "check-token" })
+  request.body = JSON.generate(json_body)
+
+  if auth
+    if cli
+      request.basic_auth("cli", "loginNOauth")
+    else
+      request.basic_auth("auth", $config['DEFAULT_SECRET'])
+    end
+  else
+    if cli
+      request.basic_auth("cli", "loginNOauth")
+    end
+  end
+
   response = Net::HTTP.start(base.hostname, $config['PORT'],
                              :timeout => $config['CALL_TIMEOUT'],
                              :use_ssl => base.scheme == "https",
@@ -248,108 +270,87 @@ def validate_token(token)
   end
   begin
     list = JSON.parse(response.body)
-    if list['error']
-      return false
-    end
-    if list['success']
-      return true
-    end
+    return list
   rescue
-    puts "\nThe server sent out an Error:".red
-    puts clean_html(response.body)
-    exit 1
+    if !auth
+      puts "\nThe server sent out an Error:".red
+      puts clean_html(response.body)
+      exit 1
+    else
+      return '{"error": "Secret is invalid"}'
+    end
   end
 end
 
-def api_caller(json_body)
-  headers = {}
-  if $config['TOKEN']
-    headers={ "bearer" => "#{$config['TOKEN']}" }
-  else
+def validate_token(token)
+  result = api_caller({"action" => "check-token" })
+  if result['error']
     return false
   end
-  base = URI.parse("#{$config['SERVERURL']}")
-  request = Net::HTTP::Post.new(base,headers)
-  request.body = JSON.generate(json_body)
-  response = Net::HTTP.start(base.hostname, $config['PORT'],
-                             :timeout => $config['CALL_TIMEOUT'],
-                             :use_ssl => base.scheme == "https",
-                             :verify_mode => OpenSSL::SSL::VERIFY_PEER,
-                             :ca_file => $config['CA_TRUST']
-                             ) do |http|
-    http.request(request)
+  if result['success']
+    return true
   end
-  begin
-    list = JSON.parse(response.body)
-    return list
-  rescue
-    puts "\nThe server sent out an Error:".red
-    puts clean_html(response.body)
-    exit 1
+end
+
+def change_user_password(user,pass,auth=false)
+  result = api_caller({"action" => "change-password", "username" => user, "password" => pass},auth)
+  if result['error']
+    return false
+  end
+  if result['success']
+    return true
   end
 end
 
+def change_username(user,new_user)
+  result = api_caller({"action" => "change-username", "username" => user, "new-username" => new_user},true)
+  if result['error']
+    return false
+  end
+  if result['success']
+    return true
+  end
+end
 
-def cross_versions
-  headers = {}
-  base = URI.parse("#{$config['SERVERURL']}")
-  request = Net::HTTP::Post.new(base,headers)
-  request.basic_auth("cli", "loginNOauth")
-  request.body = JSON.generate({"action" => "version-check" })
-  response = Net::HTTP.start(base.hostname, $config['PORT'],
-                             :timeout => $config['CALL_TIMEOUT'],
-                             :use_ssl => base.scheme == "https",
-                             :verify_mode => OpenSSL::SSL::VERIFY_PEER,
-                             :ca_file => $config['CA_TRUST']
-                             ) do |http|
-    http.request(request)
+def change_role(user,new_role)
+  result = api_caller({"action" => "change-role", "username" => user, "new-role" => new_role},true)
+  if result['error']
+    return false
   end
-  begin
-    list = JSON.parse(response.body)
-    return list
-    # if list['error']
-    #   return false
-    # end
-    # if list['success']
-    #   return true
-    # end
-  rescue
-    puts "\nThe server sent out an Error:".red
-    puts clean_html(response.body)
+  if result['success']
+    return true
+  end
+end
+
+def password_sanity(password)
+  if password.length < MIN_PASS
+    puts "New password is shorter than #{MIN_PASS} chars, please use a bigger password.".red
+    exit 1
+  end
+  if !password.test_password
+    puts "Password must be at least #{MIN_PASS} and contain at least one capital, one symbol, one number, one regular characters.".red
+    exit 1
+  end
+end
+
+def username_sanity(username)
+  if new_user.length < MIN_USER
+    puts "New Username is shorter than #{MIN_USER} chars, please use a bigger username.".red
     exit 1
   end
 end
 
+def cross_versions
+  return api_caller({"action" => "version-check" },true,true)
+end
+
 def delete_user(username)
-  headers = {}
-  if $config['TOKEN']
-    headers={ "bearer" => "#{$config['TOKEN']}" }
-  else
+  result = api_caller({"action" => "delete-user", "username" => username},true)
+  if result['error']
     return false
   end
-  base = URI.parse("#{$config['SERVERURL']}")
-  request = Net::HTTP::Post.new(base,headers)
-  request.body = JSON.generate({"action" => "delete-user", "username" => username })
-  response = Net::HTTP.start(base.hostname, $config['PORT'],
-                             :timeout => $config['CALL_TIMEOUT'],
-                             :use_ssl => base.scheme == "https",
-                             :verify_mode => OpenSSL::SSL::VERIFY_PEER,
-                             :ca_file => $config['CA_TRUST']
-                             ) do |http|
-    http.request(request)
-  end
-  begin
-    list = JSON.parse(response.body)
-    if list['error']
-      return false
-    end
-    if list['success']
-      return true
-    end
-  rescue
-    puts "\nThe server sent out an Error:".red
-    puts clean_html(response.body)
-    exit 1
+  if result['success']
+    return true
   end
 end
 
@@ -358,4 +359,4 @@ def local_version
   return
 end
 
-VERSION_SIGN="0.0.19"
+VERSION_SIGN="0.0.20"

data/lib/phoseum/phoseum-common-lib.rb

@@ -1,17 +1,51 @@
+MIN_USER = 3
+MIN_PASS = 7
+
 class String
   def remove_non_ascii(replacement='')
-      n=self.split("")
-      self.slice!(0..self.size)
-      n.each { |b|
-       if b.ord < 48 || b.ord > 57 && b.ord < 65 || b.ord > 90 && b.ord < 97 || b.ord > 122 then
-         self.concat(replacement)
-       else
-         self.concat(b)
-       end
-      }
-      self.to_s
-    end
+    n=self.split("")
+    self.slice!(0..self.size)
+    n.each { |b|
+     if b.ord < 48 || b.ord > 57 && b.ord < 65 || b.ord > 90 && b.ord < 97 || b.ord > 122 then
+       self.concat(replacement)
+     else
+       self.concat(b)
+     end
+    }
+    self.to_s
+  end
 
+  def test_password()
+    symbol = false
+    number = false
+    capital= false
+    regular= false
+    all_valid= true
+    puts "Testing #{self}"
+    n=self.split("")
+    self.slice!(0..self.size)
+    n.each { |b|
+     # test symbols
+     if b.ord > 32 && b.ord < 48 || b.ord > 58 && b.ord < 65 || b.ord > 91 && b.ord < 97 || b.ord > 123 && b.ord < 126 then
+       symbol = true
+     # test capital letters
+     elsif b.ord > 65 && b.ord < 91 then
+       capital = true
+     # test numbers
+     elsif b.ord > 47 && b.ord < 58 then
+       number = true
+     # test regular alphabet
+     elsif b.ord > 96 && b.ord < 123 then
+       regular = true
+     else
+       # com character out of the acceptable ranges
+       all_valid = false
+     end
+    }
+    if symbol && capital && number && regular && all_valid
+      return true
+    end
+  end
 end
 
 def check_string_sanity(album)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: phoseum-cli
 version: !ruby/object:Gem::Version
-  version: 0.0.19
+  version: 0.0.20
 platform: ruby
 authors:
 - Julio C Hegedus
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-18 00:00:00.000000000 Z
+date: 2020-10-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: yaml