cloudmesh-community / hid-sp18-516

Pathan, Shagufta
1 stars 4 forks source link

Assignment: Cloud and Big Data Rest Service with Swagger #6

Closed spathan1985 closed 6 years ago

spathan1985 commented 6 years ago
spathan1985 commented 6 years ago

first 2 tasks done

spathan1985 commented 6 years ago

Created new tasks

spathan1985 commented 6 years ago

@laszewsk Professor, I need to discuss few things regarding the localhost(ssh) Swagger service that I am working on. I'm having trouble with the basic authentication and also wanted to confirm if my implementation is as expected. Please let me know when we can have a call. Thanks & regards, Shagufta

spathan1985 commented 6 years ago

Created the swagger spec, generated the server-side code, implemented the REST service, implemented basic authentication and submitted the code for review.

spathan1985 commented 6 years ago

@laszewsk Professor, as per your feedback recently on Piazza (https://piazza.com/class/jbkvbp3ed3m2ez?cid=248), I have stored username/password for Basic Auth in a separate yaml file called 'config.yaml' placed under ~/server/command/flaskConnexion/creds directory. I have also tried to create a separate module for the authentication part 'auth_util.py' placed under the controllers directory for now. I simply import auth_util.py in the default_controller.py code and check for the authentication.

I am however doing 'ssh' in the subprocess.check_output command, which currently waits till I enter the password to execute the further steps, which I do not like. Going forward I would like to do the passwordless SSH and for that I will probably need another machine.

For now, request you to please take a look and provide your comments. Note: everything is checked into Github.

Thanks and regards.

spathan1985 commented 6 years ago

TODO:

spathan1985 commented 6 years ago

Updates:

spathan1985 commented 6 years ago

Today I did the following:

spathan1985 commented 6 years ago

I am able to test the service on container. Updated README with the details on how to test. Hence closing this issue.