Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Embedders widget - field for a custom server URL #114

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PrimozGodec
Copy link
Collaborator

@PrimozGodec PrimozGodec commented Sep 18, 2018

Issue

No possibility to set custom server url.

Description of changes

Adding a field and functionality for a custom URL set.

Includes
  • Code changes
  • Tests
  • Documentation

@PrimozGodec PrimozGodec force-pushed the embedders-server branch 2 times, most recently from 72c3335 to 255341b Compare September 18, 2018 10:30
@codecov-io
Copy link

codecov-io commented Sep 18, 2018

Codecov Report

Merging #114 into master will decrease coverage by 3.72%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
- Coverage    79.9%   76.18%   -3.73%     
==========================================
  Files           7        5       -2     
  Lines         622      529      -93     
  Branches       95       84      -11     
==========================================
- Hits          497      403      -94     
- Misses         97       98       +1     
  Partials       28       28

@PrimozGodec PrimozGodec changed the title [WIP] Embedders widget - field for a custom server URL Embedders widget - field for a custom server URL Sep 18, 2018
@@ -105,6 +116,15 @@ def _setup_layout(self):
EMBEDDERS_INFO[current_embedder]['description']
)

advanced_box = widgetBox(self.controlArea, 'Advanced settings')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Advanced server settings
or maybe just
Custom server

@@ -105,6 +116,15 @@ def _setup_layout(self):
EMBEDDERS_INFO[current_embedder]['description']
)

advanced_box = widgetBox(self.controlArea, 'Advanced settings')
self.use_custom_server_cb = checkBox(
advanced_box, self, 'custom_server', "User custom server",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use private server

advanced_box, self, 'custom_server', "User custom server",
callback=self.changed_server_url)
self.server_url_box = lineEdit(
advanced_box, self, 'server_url', label="Server URL: ",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this could be disabled when the above checkbox is not checked.

@matjazp
Copy link

matjazp commented Sep 21, 2018

we also need "allow insecure" option, eg. like in curl -k or --insecure (Allow connections to SSL sites without checking certs (H)). We need this if there are self signed certificates installed on the imageanalytics server.

@PrimozGodec
Copy link
Collaborator Author

@lanzagar fixed
@matjazp will try to implement. @acopar is it possible to set up a server with the self-generated certificate to test Matjaz's suggestion.

@PrimozGodec
Copy link
Collaborator Author

@matjazp it seems that hyper library does not support "allow insecure" option https://hyper.readthedocs.io/en/latest/advanced.html#ssl-tls-certificate-verification. With @acopar we discussed to just use HTTP instead of HTTPS those cases

@Corlobin
Copy link

Very useful!!

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.

5 participants