RFR: 8252767: URLConnection.setRequestProperty throws IllegalAccessError [v3]

Jaikiran Pai jpai at openjdk.java.net
Sun Sep 6 11:21:58 UTC 2020


> Can I please get a review and a sponsor for a fix for the issue reported at
> https://bugs.openjdk.java.net/browse/JDK-8252767?
> As noted in that issue, the `sun.net.www.URLConnection#setRequestProperty` is throwing a `IllegalAccessError` instead
> of a `IllegalStateException`. The commit here fixes that and includes a test which reproduces the issue and verifies
> the fix.  Would a CSR be needed for this change?

Jaikiran Pai has refreshed the contents of this pull request, and previous commits have been removed. The incremental
views will show differences compared to the previous content of the PR. The pull request contains one new commit since
the last revision:

  8252767: URLConnection.setRequestProperty throws IllegalAccessError
  Summary: Throw an IllegalStateException from sun.net.www.URLConnection#setRequestProperty when already connected
  Reviewed-By:

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/26/files
  - new: https://git.openjdk.java.net/jdk/pull/26/files/faf32fcf..a819f50c

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=26&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=26&range=01-02

  Stats: 3 lines in 1 file changed: 1 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/26.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/26/head:pull/26

PR: https://git.openjdk.java.net/jdk/pull/26


More information about the net-dev mailing list