Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8245984

Remove terminally deprecated Solaris-specific SO_FLOW_SLA socket option

    XMLWordPrintable

    Details

    • Type: CSR
    • Status: Closed
    • Priority: P3
    • Resolution: Approved
    • Fix Version/s: 15
    • Component/s: core-libs
    • Labels:
      None
    • Subcomponent:
    • Compatibility Kind:
      source, binary
    • Compatibility Risk:
      low
    • Compatibility Risk Description:
      The option was only supported on Solaris. Code that referenced this option will need to be removed. It is unlikely that an application not explicitly designed to run on Solaris would reference this option.
    • Interface Kind:
      Java API
    • Scope:
      JDK

      Description

      Summary

      Remove jdk.net.ExtendedSocketOptions.SO_FLOW_SLA, SocketFlow and SocketFlow.Status. The Solaris and JDK-specific SO_FLOW_SLA socket option was terminally deprecated in JDK 14 (JDK-8235194) in preparation for the removal of the Solaris port. JEP 381 has already removed the implementation from JDK 15. The API elements (socket option and supporting classes) can now be removed.

      Problem

      The specification for ExtendedSocketOptions.SO_FLOW_SLA will need to be removed along with the option itself.

      The class level API documentation of NetworkPermission describes the permission targets required to get and set the ExtendedSocketOptions.SO_FLOW_SLA option. The description also currently marks these permission targets as being deprecated. The description of the permission targets will need to be removed from the class level API documentation of NetworkPermission.

      Solution

      Remove all related specification for ExtendedSocketOptions.SO_FLOW_SLA.

      Specification

      Remove jdk.net.ExtendedSocketOption.SO_FLOW_SLA

      Remove jdk.net.SocketFlow

      Remove jdk.net.SocketFlow.Status

      Remove the list of targets from jdk.net.NetworkPermission

           /**
            * Represents permission to access the extended networking capabilities
            * defined in the jdk.net package. These permissions contain a target
            * name, but no actions list. Callers either possess the permission or not.
      -     * <p>
      -     * The following targets are defined:
      -     *
      -     * <table class="striped"><caption style="display:none">permission target name,
      -     *  what the target allows,and associated risks</caption>
      -     * <thead>
      -     * <tr>
      -     *   <th scope="col">Permission Target Name</th>
      -     *   <th scope="col">What the Permission Allows</th>
      -     *   <th scope="col">Risks of Allowing this Permission</th>
      -     * </tr>
      -     * </thead>
      -     * <tbody>
      -     * <tr>
      -     *   <th scope="row">setOption.SO_FLOW_SLA</th>
      -     *   <td>set the {@link ExtendedSocketOptions#SO_FLOW_SLA SO_FLOW_SLA} option
      -     *       on any socket that supports it</td>
      -     *   <td>allows caller to set a higher priority or bandwidth allocation
      -     *       to sockets it creates, than they might otherwise be allowed.
      -     *       This permission is deprecated.</td>
      -     * </tr>
      -     * <tr>
      -     *   <th scope="row">getOption.SO_FLOW_SLA</th>
      -     *   <td>retrieve the {@link ExtendedSocketOptions#SO_FLOW_SLA SO_FLOW_SLA}
      -     *       setting from any socket that supports the option</td>
      -     *   <td>allows caller access to SLA information that it might not
      -     *       otherwise have. This permission is deprecated.</td>
      -     * </tr>
      -     * </tbody>
      -     * </table>
            *
            * @see jdk.net.ExtendedSocketOptions
            *
            * @since 1.8
            */
      
           public final class NetworkPermission extends BasicPermission {

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              pconcannon Patrick Concannon (Inactive)
              Reporter:
              alanb Alan Bateman
              Reviewed By:
              Alan Bateman, Chris Hegarty, Daniel Fuchs
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: