Skip to content

Commit

Permalink
Add deprecation warnings.
Browse files Browse the repository at this point in the history
Should pass a SocketFactory around instead of SSL properties.
  • Loading branch information
amolenaar committed May 14, 2016
1 parent 120afbe commit d19accd
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
3 changes: 3 additions & 0 deletions src/fitnesse/FitNesseContext.java
Expand Up @@ -50,8 +50,11 @@ public class FitNesseContext {
public final RecentChanges recentChanges;
public final Logger logger;
public final Authenticator authenticator;
@Deprecated // ("Change to SocketFactory instead")
public final boolean useHTTPS;
@Deprecated // ("Change to SocketFactory instead")
public String sslParameterClassName;
@Deprecated // ("Change to SocketFactory instead")
public final boolean sslClientAuth;
private final Properties properties;

Expand Down
4 changes: 3 additions & 1 deletion src/fitnesse/socketservice/SocketFactory.java
Expand Up @@ -19,6 +19,7 @@
import javax.net.ssl.SSLSocketFactory;


@Deprecated // Change to proper class / interface
public final class SocketFactory {
private static final Logger LOG = Logger.getLogger(SocketFactory.class.getName());

Expand All @@ -33,6 +34,7 @@ public static ServerSocket createServerSocket(int port) throws IOException {
return new ServerSocket(port);
}

@Deprecated
public static ServerSocket createSslServerSocket(int port, boolean needClientAuth, String sslParameterClassName) throws IOException {
ServerSocket socket;
LOG.log(Level.FINER, "Creating SSL socket on port: " + port);
Expand Down Expand Up @@ -179,4 +181,4 @@ public static void printSocketInfo(Socket theSocket) {
}
}
}
}
}

0 comments on commit d19accd

Please sign in to comment.