-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Fixes #9288 - Jetty 12 - Use oej.http.HttpCookie in jetty-client. #9289
Conversation
7a3d7e2
to
7357322
Compare
7111a7e
to
efe58fe
Compare
jetty-core/jetty-http/src/main/java/org/eclipse/jetty/http/HttpCookieStore.java
Show resolved
Hide resolved
jetty-core/jetty-http/src/main/java/org/eclipse/jetty/http/HttpCookieStore.java
Show resolved
Hide resolved
* Replaced usages of java.net.HttpCookie with oej.http.HttpCookie. * Moved server-side only methods from HttpCookie to HttpCookieUtils. * Introduced and implemented oej.http.HttpCookieStore. * Removed now obsolete oej.util.HttpCookieStore. Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
efe58fe
to
4b7d274
Compare
@sbordet please don't force push. There is no need for that as we squash when we merge. |
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.
A few niggles, but otherwise I can't see any problems.
jetty-core/jetty-http/src/main/java/org/eclipse/jetty/http/HttpCookieStore.java
Show resolved
Hide resolved
jetty-core/jetty-http/src/main/java/org/eclipse/jetty/http/HttpCookieStore.java
Show resolved
Hide resolved
jetty-core/jetty-http/src/main/java/org/eclipse/jetty/http/HttpCookieStore.java
Show resolved
Hide resolved
jetty-core/jetty-http/src/main/java/org/eclipse/jetty/http/HttpCookieStore.java
Show resolved
Hide resolved
jetty-core/jetty-http/src/main/java/org/eclipse/jetty/http/HttpCookieStore.java
Show resolved
Hide resolved
jetty-core/jetty-client/src/main/java/org/eclipse/jetty/client/transport/HttpDestination.java
Show resolved
Hide resolved
jetty-core/jetty-client/src/main/java/org/eclipse/jetty/client/transport/HttpDestination.java
Outdated
Show resolved
Hide resolved
jetty-core/jetty-client/src/main/java/org/eclipse/jetty/client/transport/HttpDestination.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
…x-documentation-operations-logging * upstream/jetty-12.0.x: Jetty 12: Cleanup `StatisticsHandler` (jetty#9291) Fixes jetty#9288 - Jetty 12 - Use oej.http.HttpCookie in jetty-client. (jetty#9289) Redo Handler renaming (jetty#9318)
Signed-off-by: Simone Bordet simone.bordet@gmail.com