Follow-up to OctoPrint upload: ignore certificate revocation checks

6b03b66167

Renamed the new "printhost_ignore_check" option to "printhost_ssl_ignore_revoke"

Improved the Physical Printers dialog in regard to the new option checkbox
(added tooltip to the checkbox, moved it to the end of options).

Disabled the host_xxx options at the command line interface, they no
more work after these options were separated to Physical Printers profiles.

Little refactoring of Http.cpp/hpp, OctoPrint.cpp/hpp
Private local variables prefixed with m_, some renaming for clarity.

src/slic3r/Utils/Http.cpp

@@ -491,7 +491,9 @@ Http& Http::form_add_file(const std::string &name, const fs::path &path, const s
 	return *this;
 }
 
-Http& Http::revoke_best_effort(bool set)
+// Tells libcurl to ignore certificate revocation checks in case of missing or offline distribution points for those SSL backends where such behavior is present. 
+// This option is only supported for Schannel (the native Windows SSL library).
+Http& Http::ssl_revoke_best_effort(bool set)
 {
 	if(p && set){
 		::curl_easy_setopt(p->curl, CURLOPT_SSL_OPTIONS, CURLSSLOPT_REVOKE_BEST_EFFORT);

src/slic3r/Utils/Http.hpp

@@ -80,7 +80,11 @@ public:
 	// Same as above except also override the file's filename with a custom one
 	Http& form_add_file(const std::string &name, const boost::filesystem::path &path, const std::string &filename);
 
-	Http& revoke_best_effort(bool set);
+#ifdef WIN32
+	// Tells libcurl to ignore certificate revocation checks in case of missing or offline distribution points for those SSL backends where such behavior is present. 
+	// This option is only supported for Schannel (the native Windows SSL library).
+	Http& ssl_revoke_best_effort(bool set);
+#endif // WIN32
 
 	// Set the file contents as a POST request body.
 	// The data is used verbatim, it is not additionally encoded in any way.

src/slic3r/Utils/OctoPrint.cpp

@@ -23,10 +23,10 @@ namespace pt = boost::property_tree;
 namespace Slic3r {
 
 OctoPrint::OctoPrint(DynamicPrintConfig *config) :
-    host(config->opt_string("print_host")),
-    apikey(config->opt_string("printhost_apikey")),
-    cafile(config->opt_string("printhost_cafile")),
-    ignore_checks(config->opt_bool("printhost_ignore_check"))
+    m_host(config->opt_string("print_host")),
+    m_apikey(config->opt_string("printhost_apikey")),
+    m_cafile(config->opt_string("printhost_cafile")),
+    m_ssl_revoke_best_effort(config->opt_bool("printhost_ssl_ignore_revoke"))
 {}
 
 const char* OctoPrint::get_name() const { return "OctoPrint"; }
@@ -75,7 +75,7 @@ bool OctoPrint::test(wxString &msg) const
             }
         })
 #ifdef WIN32
-        .revoke_best_effort(ignore_checks)
+        .ssl_revoke_best_effort(m_ssl_revoke_best_effort)
 #endif
         .perform_sync();
 
@@ -142,7 +142,7 @@ bool OctoPrint::upload(PrintHostUpload upload_data, ProgressFn prorgess_fn, Erro
             }
         })
 #ifdef WIN32
-        .revoke_best_effort(ignore_checks)
+        .ssl_revoke_best_effort(m_ssl_revoke_best_effort)
 #endif
         .perform_sync();
 
@@ -156,31 +156,31 @@ bool OctoPrint::validate_version_text(const boost::optional<std::string> &versio
 
 void OctoPrint::set_auth(Http &http) const
 {
-    http.header("X-Api-Key", apikey);
+    http.header("X-Api-Key", m_apikey);
 
-    if (! cafile.empty()) {
-        http.ca_file(cafile);
+    if (!m_cafile.empty()) {
+        http.ca_file(m_cafile);
     }
 }
 
 std::string OctoPrint::make_url(const std::string &path) const
 {
-    if (host.find("http://") == 0 || host.find("https://") == 0) {
-        if (host.back() == '/') {
-            return (boost::format("%1%%2%") % host % path).str();
+    if (m_host.find("http://") == 0 || m_host.find("https://") == 0) {
+        if (m_host.back() == '/') {
+            return (boost::format("%1%%2%") % m_host % path).str();
         } else {
-            return (boost::format("%1%/%2%") % host % path).str();
+            return (boost::format("%1%/%2%") % m_host % path).str();
         }
     } else {
-        return (boost::format("http://%1%/%2%") % host % path).str();
+        return (boost::format("http://%1%/%2%") % m_host % path).str();
     }
 }
 
 SL1Host::SL1Host(DynamicPrintConfig *config) : 
     OctoPrint(config),
-    authorization_type(dynamic_cast<const ConfigOptionEnum<AuthorizationType>*>(config->option("printhost_authorization_type"))->value),
-    username(config->opt_string("printhost_user")),
-    password(config->opt_string("printhost_password"))
+    m_authorization_type(dynamic_cast<const ConfigOptionEnum<AuthorizationType>*>(config->option("printhost_authorization_type"))->value),
+    m_username(config->opt_string("printhost_user")),
+    m_password(config->opt_string("printhost_password"))
 {
 }
 
@@ -206,12 +206,12 @@ bool SL1Host::validate_version_text(const boost::optional<std::string> &version_
 
 void SL1Host::set_auth(Http &http) const
 {
-    switch (authorization_type) {
+    switch (m_authorization_type) {
     case atKeyPassword:
         http.header("X-Api-Key", get_apikey());
         break;
     case atUserPassword:
-        http.auth_digest(username, password);
+        http.auth_digest(m_username, m_password);
         break;
     }
 
@@ -223,9 +223,9 @@ void SL1Host::set_auth(Http &http) const
 // PrusaLink
 PrusaLink::PrusaLink(DynamicPrintConfig* config) :
     OctoPrint(config),
-    authorization_type(dynamic_cast<const ConfigOptionEnum<AuthorizationType>*>(config->option("printhost_authorization_type"))->value),
-    username(config->opt_string("printhost_user")),
-    password(config->opt_string("printhost_password"))
+    m_authorization_type(dynamic_cast<const ConfigOptionEnum<AuthorizationType>*>(config->option("printhost_authorization_type"))->value),
+    m_username(config->opt_string("printhost_user")),
+    m_password(config->opt_string("printhost_password"))
 {
 }
 
@@ -250,12 +250,12 @@ bool PrusaLink::validate_version_text(const boost::optional<std::string>& versio
 
 void PrusaLink::set_auth(Http& http) const
 {
-    switch (authorization_type) {
+    switch (m_authorization_type) {
     case atKeyPassword:
         http.header("X-Api-Key", get_apikey());
         break;
     case atUserPassword:
-        http.auth_digest(username, password);
+        http.auth_digest(m_username, m_password);
         break;
     }
 

src/slic3r/Utils/OctoPrint.hpp

@@ -29,18 +29,18 @@ public:
     bool has_auto_discovery() const override { return true; }
     bool can_test() const override { return true; }
     bool can_start_print() const override { return true; }
-    std::string get_host() const override { return host; }
-    const std::string& get_apikey() const { return apikey; }
-    const std::string& get_cafile() const { return cafile; }
+    std::string get_host() const override { return m_host; }
+    const std::string& get_apikey() const { return m_apikey; }
+    const std::string& get_cafile() const { return m_cafile; }
 
 protected:
     virtual bool validate_version_text(const boost::optional<std::string> &version_text) const;
 
 private:
-    std::string host;
-    std::string apikey;
-    std::string cafile;
-    bool        ignore_checks;
+    std::string m_host;
+    std::string m_apikey;
+    std::string m_cafile;
+    bool        m_ssl_revoke_best_effort;
 
     virtual void set_auth(Http &http) const;
     std::string make_url(const std::string &path) const;
@@ -65,10 +65,10 @@ private:
     void set_auth(Http &http) const override;
 
     // Host authorization type.
-    AuthorizationType authorization_type;
+    AuthorizationType m_authorization_type;
     // username and password for HTTP Digest Authentization (RFC RFC2617)
-    std::string username;
-    std::string password;
+    std::string m_username;
+    std::string m_password;
 };
 
 class PrusaLink : public OctoPrint
@@ -90,10 +90,10 @@ private:
     void set_auth(Http& http) const override;
 
     // Host authorization type.
-    AuthorizationType authorization_type;
+    AuthorizationType m_authorization_type;
     // username and password for HTTP Digest Authentization (RFC RFC2617)
-    std::string username;
-    std::string password;
+    std::string m_username;
+    std::string m_password;
 };
 
 }