Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@larso
Copy link

@larso larso commented Sep 18, 2015

Minor update adding support to nt_import for using https towards a NicTool server. Added:

  • use-https commandline option to nt_import
  • support for a $nt_https parameter in NicToolServer::Import::Base::nt_connect function

@msimerson
Copy link
Collaborator

/me wonders if we shouldn't deprecate http and require https for those connections.

@larso
Copy link
Author

larso commented Sep 18, 2015

It's probably a good idea to run this over https :-)
But in order for such a deprecation to succeed, someone has to ensure that all of the codebase supports https. I have only submitted patches fixing parts of the code where this issue has occured for us.

msimerson added a commit that referenced this pull request Sep 18, 2015
Added https support for nt_import and NicToolServer::Import::Base
@msimerson msimerson merged commit da256e9 into NicTool:master Sep 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants