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
SparkServer was updated to be runnable on http and https protocols at… #329
Conversation
… same time. Api call was added to SparkBase to support that. Ability to specify special ssl port and justSsl flag were added.
When is supposed this to be merged? I need this. |
protected static final String DEFAULT_ACCEPT_TYPE = "*/*"; | ||
|
||
protected static boolean initialized = false; | ||
|
||
protected static int port = SPARK_DEFAULT_PORT; | ||
protected static int sslPort = SPARK_DEFAULT_PORT; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should't sslPort have the value of SPARK_DEFAULT_PORT_SSL?
+1 |
Conflicts: src/main/java/spark/SparkBase.java src/main/java/spark/webserver/SparkServer.java
Pull request was synced with latest code base. |
Where exactly is it synchronized? In the master code base I can not find the string 'SPARK_DEFAULT_PORT_SSL' neither the 'sslPort' variable? |
It was better to say that it is redone. @georgeyanev Please see commit e1b6144 |
@gimlet2 Is this feature is available in 2.5.3 release? I need this feature ASAP. thanks |
Is this issue still relevant now when there's an instance API? Service http = ignite()
Service https = ignite()
http.port(1234)
https.port(1235)
https.secure(... |
As per mentioned, I think this should be taken care of by the instance-api, or in a layer in front of Spark. |
… same time. Api call was added to SparkBase to support that. Ability to specify special ssl port and justSsl flag were added.