markt-asf commented on PR #646:
URL: https://github.com/apache/tomcat/pull/646#issuecomment-1685427252
Thanks for the review. I checked the various getParameter() methods and
found a couple I thought needed a try/catch block. The remainder were all used
in webapps or in APIs called by appli
tutufool opened a new issue, #47:
URL: https://github.com/apache/tomcat-jakartaee-migration/issues/47
Hi,
I'd like to know when a specific package is replaced (like javax.xml.bind
replaced by jakarta.xml.bind), shall we need introduce some 3rd party
dependency?
like:
j
michael-o commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1299773529
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
markt-asf commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1299927091
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
markt-asf commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1299935009
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
markt-asf commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1299935328
##
webapps/docs/changelog.xml:
##
@@ -113,6 +113,12 @@
provided error code during error page processing rather than assuming
an
error code of 500. (ma
markt-asf commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1299936723
##
webapps/docs/config/http.xml:
##
@@ -148,20 +148,17 @@
files) obtained from the query string and, for POST requests, the request
body if the content typ
michael-o commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1299969203
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
michael-o commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1299969607
##
webapps/docs/config/http.xml:
##
@@ -148,20 +148,17 @@
files) obtained from the query string and, for POST requests, the request
body if the content typ
markt-asf commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1300034970
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
michael-o commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1300043089
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
michael-o commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1300652715
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,98 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
michael-o commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1300652715
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,98 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
markt-asf commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1300656508
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,98 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
michael-o commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1300658108
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,98 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
markt-asf commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1300661077
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,98 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
sergmain opened a new pull request, #647:
URL: https://github.com/apache/tomcat/pull/647
Replaced synchronized with StampedLock as preparation for virtual threads
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
tutufool commented on issue #47:
URL:
https://github.com/apache/tomcat-jakartaee-migration/issues/47#issuecomment-1687326690
cool, thanks a lot!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
michael-o commented on code in PR #646:
URL: https://github.com/apache/tomcat/pull/646#discussion_r1301062237
##
java/org/apache/tomcat/util/http/InvalidParameterException.java:
##
@@ -0,0 +1,98 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
markt-asf commented on PR #647:
URL: https://github.com/apache/tomcat/pull/647#issuecomment-1688571304
What is the justification for this change? Synchronization is not
automatically an issue for virtual threads. I'm not seeing anything in this
code that would be unfriendly to virtual threa
markt-asf merged PR #646:
URL: https://github.com/apache/tomcat/pull/646
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.o
sergmain commented on PR #647:
URL: https://github.com/apache/tomcat/pull/647#issuecomment-1689065406
From my point of view there are 2 approaches - analyze every synchronized or
remove every synchronized without analyzing. My pull-request is with second
approach.
--
This is an automated
markt-asf commented on PR #647:
URL: https://github.com/apache/tomcat/pull/647#issuecomment-1689066140
Synchronized blocks only need to be replaced if they contain blocking
operations. This one clearly doesn't.
--
This is an automated message from the Apache Git Service.
To respond to the
markt-asf closed pull request #647: Replaced synchronized with StampedLock
URL: https://github.com/apache/tomcat/pull/647
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To uns
markt-asf merged PR #6:
URL: https://github.com/apache/tomcat-connectors/pull/6
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.a
markt-asf commented on PR #5:
URL: https://github.com/apache/tomcat-connectors/pull/5#issuecomment-1705720654
The proposed patch does not address the root cause of the bug which can be
found at
https://github.com/apache/tomcat-connectors/blob/main/native/apache-2.0/mod_jk.c#L3016
I h
markt-asf closed pull request #5: Fix bug #65901 to correct HTTP 401 response
for HEAD request
URL: https://github.com/apache/tomcat-connectors/pull/5
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
xxeol2 opened a new pull request, #648:
URL: https://github.com/apache/tomcat/pull/648
This Pull Request aims to reduce code duplication in the `getSession` method
of the `Request` class.
The refactoring has been inspired by the format used in the `reset` method
of `java.org.apache.catal
aooohan commented on code in PR #648:
URL: https://github.com/apache/tomcat/pull/648#discussion_r1315332131
##
java/org/apache/catalina/connector/Request.java:
##
@@ -2288,12 +2288,7 @@ public String getServletPath() {
*/
@Override
public HttpSession getSession()
xxeol2 commented on code in PR #648:
URL: https://github.com/apache/tomcat/pull/648#discussion_r1315352368
##
java/org/apache/catalina/connector/Request.java:
##
@@ -2288,12 +2288,7 @@ public String getServletPath() {
*/
@Override
public HttpSession getSession()
xxeol2 opened a new pull request, #649:
URL: https://github.com/apache/tomcat/pull/649
In the current implementation of the getSession method in the RequestFacade
class, the checkFacade logic was redundantly executed twice.
I improved this by eliminating the redundancy.
--
This is an a
aooohan merged PR #648:
URL: https://github.com/apache/tomcat/pull/648
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
aooohan commented on PR #648:
URL: https://github.com/apache/tomcat/pull/648#issuecomment-1706002731
Thanks for the PR. ; )
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
greeng00se opened a new pull request, #650:
URL: https://github.com/apache/tomcat/pull/650
hello 😄
I was looking at the code for the Request class in coyote.
Removed unnecessary brackets for consistency with the other code.
e.g. `contentLength = (clB == null || clB.isNull())
wonyongChoi05 opened a new pull request, #651:
URL: https://github.com/apache/tomcat/pull/651
Simplified the SSL certificate population logic by removing the unnecessary
null check and using an ArrayList for dynamic array management. The code is now
more concise and readable, and it efficie
aooohan closed pull request #650: Removed unnecessary brackets
URL: https://github.com/apache/tomcat/pull/650
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-
aooohan commented on PR #649:
URL: https://github.com/apache/tomcat/pull/649#issuecomment-1706095437
This change doesn't make any sense.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the sp
aooohan closed pull request #649: Eliminating duplicate execution of
checkFacade logic
URL: https://github.com/apache/tomcat/pull/649
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific com
aooohan commented on PR #650:
URL: https://github.com/apache/tomcat/pull/650#issuecomment-1706094596
This change doesn't make any sense.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specif
aooohan commented on PR #651:
URL: https://github.com/apache/tomcat/pull/651#issuecomment-1706113147
This change doesn't make any sense.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specif
aooohan closed pull request #651: Refactor SSL certificate population method
URL: https://github.com/apache/tomcat/pull/651
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To u
markt-asf commented on PR #650:
URL: https://github.com/apache/tomcat/pull/650#issuecomment-1706269180
The change is trivial but it makes sense to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
markt-asf merged PR #650:
URL: https://github.com/apache/tomcat/pull/650
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.o
markt-asf commented on PR #649:
URL: https://github.com/apache/tomcat/pull/649#issuecomment-1706282024
The change makes sense to me. It removes a duplicate call to `checkFacade()`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
markt-asf merged PR #649:
URL: https://github.com/apache/tomcat/pull/649
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.o
markt-asf commented on PR #651:
URL: https://github.com/apache/tomcat/pull/651#issuecomment-1706340615
The switch to using a wildcard import would need to be reverted as would the
changes to blank lines between methods.
The null check is necessary in some form and it is more concise than
wonyongChoi05 commented on PR #651:
URL: https://github.com/apache/tomcat/pull/651#issuecomment-1706354873
> The switch to using a wildcard import would need to be reverted as would
the changes to blank lines between methods. The null check is necessary in some
form and it is more concise t
iamjooon2 opened a new pull request, #652:
URL: https://github.com/apache/tomcat/pull/652
Hello!
i deleted needless brackets for consistency with the other code.
Thx :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
woosung1223 opened a new pull request, #653:
URL: https://github.com/apache/tomcat/pull/653
Hi there!
I found unnecessary code in the Response class.
Currently, it seems that only headers of Content-Length or Content-Type are
called Special Headers.
However, as you can s
xxeol2 opened a new pull request, #654:
URL: https://github.com/apache/tomcat/pull/654
This PR is closely related to https://github.com/apache/tomcat/pull/649.
Prevent double execution of the checkFacade() logic when invoking the
isCommitted() method.
thanks !! 😁
--
This is an a
ingpyo opened a new pull request, #655:
URL: https://github.com/apache/tomcat/pull/655
Hello,
I have fixed a typo in the Tomcat codebase by changing "CookiePreocessor" to
"CookieProcessor" for better code clarity and correctness.
--
This is an automated message from the Apache Git Serv
xxeol2 commented on PR #654:
URL: https://github.com/apache/tomcat/pull/654#issuecomment-1706706294
I'm curious if following code changes would introduce any side effects.
### Origin
```java
public ServletOutputStream getOutputStream() throws IOException {
checkFacade();
markt-asf commented on PR #652:
URL: https://github.com/apache/tomcat/pull/652#issuecomment-1706738930
We have started to get a lot of PRs removing single instances of unnecessary
parentheses. While we are generally in favour of this sort of clean-up, one PR
per instance is not scaleable an
markt-asf merged PR #655:
URL: https://github.com/apache/tomcat/pull/655
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.o
markt-asf commented on PR #653:
URL: https://github.com/apache/tomcat/pull/653#issuecomment-1706758159
Those checks were added for performance reasons. They were significantly
quicker than calling `checkSpecialHeader()`. Unless a benchmark (as a unit
test) is provided that demonstrates that
markt-asf merged PR #654:
URL: https://github.com/apache/tomcat/pull/654
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.o
markt-asf commented on PR #654:
URL: https://github.com/apache/tomcat/pull/654#issuecomment-1706766432
Nice catch. Tx for the PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comm
xxeol2 commented on PR #654:
URL: https://github.com/apache/tomcat/pull/654#issuecomment-1706772038
@markt-asf
I was wondering if I shouldn't avoid duplicate calls to `checkFacade()` in
the `getOutputStream()` and `getWriter()` methods!
https://github.com/apache/tomcat/pull/654#issuecomm
woosung1223 closed pull request #653: unnecessary work is done before
determining if it is a special header
URL: https://github.com/apache/tomcat/pull/653
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to g
woosung1223 commented on PR #653:
URL: https://github.com/apache/tomcat/pull/653#issuecomment-1706779668
> Those checks were added for performance reasons. They were significantly
quicker than calling checkSpecialHeader(). Unless a benchmark (as a unit test)
is provided that demonstrates th
iamjooon2 closed pull request #652: Remove unnecessary brackets
URL: https://github.com/apache/tomcat/pull/652
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e
greeng00se opened a new pull request, #656:
URL: https://github.com/apache/tomcat/pull/656
Existing code calls getConfiguredSessionCookieName even if the context is
empty.
In getConfiguredSessionCookieName, only act to get the session cookie if the
context is non-null.
I think
greeng00se closed pull request #656: Optimization when call
getSessionCookieName & getSessionUriParamName
URL: https://github.com/apache/tomcat/pull/656
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
markt-asf commented on PR #654:
URL: https://github.com/apache/tomcat/pull/654#issuecomment-1706867922
Those refactorings look OK. Please submit another PR for those.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
xxeol2 opened a new pull request, #657:
URL: https://github.com/apache/tomcat/pull/657
https://github.com/apache/tomcat/pull/654#issuecomment-1706706294
This was discussed in the comments above!
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
greeng00se commented on PR #656:
URL: https://github.com/apache/tomcat/pull/656#issuecomment-1706941221
I didn't think about when neither priority 1 nor 2 is present in pr, sorry.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
Jaeyoung22 opened a new pull request, #658:
URL: https://github.com/apache/tomcat/pull/658
Although I totally agree that there could be some unnecessary parentheses to
aid the readability of the code, I think that the unity of the code is still
important for readability.
### Conditio
parkmuhyeun opened a new pull request, #659:
URL: https://github.com/apache/tomcat/pull/659
## Description
This PR made the following two modifications
- Unify constant delimiters
- Refactoring for better readability
## Explanation
### Unify constant delimiters
markt-asf commented on code in PR #657:
URL: https://github.com/apache/tomcat/pull/657#discussion_r1316282831
##
java/org/apache/catalina/connector/ResponseFacade.java:
##
@@ -111,23 +111,19 @@ public String getCharacterEncoding() {
@Override
public ServletOutputStre
markt-asf commented on code in PR #659:
URL: https://github.com/apache/tomcat/pull/659#discussion_r1316303250
##
java/org/apache/catalina/manager/StatusTransformer.java:
##
@@ -375,33 +375,33 @@ protected static void writeProcessorState(PrintWriter
writer,
switch (st
parkmuhyeun commented on PR #659:
URL: https://github.com/apache/tomcat/pull/659#issuecomment-1707218270
I reverted the stage constants back and modified the switch statements to
use the constants that were originally there!
--
This is an automated message from the Apache Git Service.
To
greeng00se opened a new pull request, #660:
URL: https://github.com/apache/tomcat/pull/660
### Description
Optimization when call getSessionCookieName & getSessionUriParamName
### Explanation
Existing code calls getConfiguredSessionCookieName even if the context is
empty
greeng00se commented on PR #650:
URL: https://github.com/apache/tomcat/pull/650#issuecomment-1707230523
@markt-asf thx for merge my first contribution 😄
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
greeng00se commented on PR #656:
URL: https://github.com/apache/tomcat/pull/656#issuecomment-1707231871
sorry. Case 3 is not considered.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specif
markt-asf commented on PR #660:
URL: https://github.com/apache/tomcat/pull/660#issuecomment-1707256734
Saving a single function call is unlikely to make any difference. I'd expect
the compiler to optimise that.
Changes that reduce duplication and/or improve the readability of the code
greeng00se commented on PR #660:
URL: https://github.com/apache/tomcat/pull/660#issuecomment-1707265409
@markt-asf
Thanks for the nice comment.
I hadn't considered that partial compiler optimisation.
I agree, readable code should be a priority and I moved the null-check logic
to a
xxeol2 commented on code in PR #657:
URL: https://github.com/apache/tomcat/pull/657#discussion_r1316480032
##
java/org/apache/catalina/connector/ResponseFacade.java:
##
@@ -111,23 +111,19 @@ public String getCharacterEncoding() {
@Override
public ServletOutputStream
Jaeyoung22 closed pull request #658: Unify conditional statement format for
some constants in http1 package
URL: https://github.com/apache/tomcat/pull/658
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to g
Jaeyoung22 closed pull request #658: Unify conditional statement format for
some constants in http1 package
URL: https://github.com/apache/tomcat/pull/658
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to g
Jaeyoung22 opened a new pull request, #661:
URL: https://github.com/apache/tomcat/pull/661
Although I totally agree that there could be some unnecessary parentheses to
aid the readability of the code, I think that the unity of the code is still
important for readability.
### Conditio
parkmuhyeun commented on PR #659:
URL: https://github.com/apache/tomcat/pull/659#issuecomment-1707461772
Oh, sorry. I missed that one! I've fixed it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
iamjooon2 closed pull request #652: Remove unnecessary brackets
URL: https://github.com/apache/tomcat/pull/652
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e
iamjooon2 commented on PR #652:
URL: https://github.com/apache/tomcat/pull/652#issuecomment-1707507180
@markt-asf I just saw [!This PR](https://github.com/apache/tomcat/pull/650)
and removed some brackets too
Regret for comment 'Automated tools'😕
--
This is an automated message fro
Jaeyoung22 commented on PR #658:
URL: https://github.com/apache/tomcat/pull/658#issuecomment-1707535138
This Pull Request UNINTENTIONALLY sended because of github incident :( So I
closed this PR and resend.
--
This is an automated message from the Apache Git Service.
To respond to the mes
wonyongChoi05 opened a new pull request, #662:
URL: https://github.com/apache/tomcat/pull/662
Hello! Thank you for the PR. Upon reviewing these changes, I've noticed a
significant improvement in code readability. This enhanced readability stands
out and will make a substantial difference in
markt-asf merged PR #657:
URL: https://github.com/apache/tomcat/pull/657
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.o
markt-asf commented on code in PR #661:
URL: https://github.com/apache/tomcat/pull/661#discussion_r1316875467
##
java/org/apache/coyote/http11/filters/ChunkedInputFilter.java:
##
@@ -574,7 +574,7 @@ private boolean parseHeader() throws IOException {
}
markt-asf commented on code in PR #659:
URL: https://github.com/apache/tomcat/pull/659#discussion_r1316887097
##
java/org/apache/coyote/http11/Constants.java:
##
@@ -106,7 +106,7 @@ public final class Constants {
public static final String KEEP_ALIVE_HEADER_VALUE_TOKEN = "k
markt-asf commented on code in PR #660:
URL: https://github.com/apache/tomcat/pull/660#discussion_r1316890952
##
java/org/apache/catalina/util/SessionConfig.java:
##
@@ -49,38 +42,30 @@ public static String getSessionCookieName(Context context) {
* @return the parameter na
greeng00se commented on code in PR #660:
URL: https://github.com/apache/tomcat/pull/660#discussion_r1316896601
##
java/org/apache/catalina/util/SessionConfig.java:
##
@@ -49,38 +42,30 @@ public static String getSessionCookieName(Context context) {
* @return the parameter n
greeng00se commented on code in PR #660:
URL: https://github.com/apache/tomcat/pull/660#discussion_r1316896601
##
java/org/apache/catalina/util/SessionConfig.java:
##
@@ -49,38 +42,30 @@ public static String getSessionCookieName(Context context) {
* @return the parameter n
parkmuhyeun commented on PR #659:
URL: https://github.com/apache/tomcat/pull/659#issuecomment-1707887311
TransferENCODING wasn't mentioned, so I didn't recognize it. I reverted all
the stuff about constant names!
--
This is an automated message from the Apache Git Service.
To respond to t
markt-asf commented on code in PR #662:
URL: https://github.com/apache/tomcat/pull/662#discussion_r1316892872
##
java/org/apache/coyote/Request.java:
##
@@ -16,19 +16,8 @@
*/
package org.apache.coyote;
-import java.io.IOException;
-import java.io.UnsupportedEncodingExceptio
apptie opened a new pull request, #663:
URL: https://github.com/apache/tomcat/pull/663
Hello!
While learning about Tomcat, I see something that I think I can contribute
to, so I write this PR.
## Explanation
There are two commits here:
- Remove the FQCN when creating
sosow0212 opened a new pull request, #664:
URL: https://github.com/apache/tomcat/pull/664
While seeing the code in the Coyote project, I came across this code
snippet:
`if (!(chr == Constants.SP || chr == Constants.HT))`.
It appears to be challenging to read, so I enhanced its rea
markt-asf commented on code in PR #663:
URL: https://github.com/apache/tomcat/pull/663#discussion_r1316972689
##
java/org/apache/catalina/startup/Tomcat.java:
##
@@ -1303,7 +1303,7 @@ public static void main(String[] args) throws Exception {
break;
markt-asf commented on PR #661:
URL: https://github.com/apache/tomcat/pull/661#issuecomment-1707956244
See #663 - a single PR would be preferred.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to t
markt-asf commented on PR #664:
URL: https://github.com/apache/tomcat/pull/664#issuecomment-1707957682
See #661 - a single PR would be preferred.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to t
wonyongChoi05 commented on code in PR #662:
URL: https://github.com/apache/tomcat/pull/662#discussion_r1317009172
##
java/org/apache/coyote/Request.java:
##
@@ -853,24 +846,22 @@ public boolean isProcessing() {
* @param contentType a content type header
*/
priva
iamjooon2 commented on PR #652:
URL: https://github.com/apache/tomcat/pull/652#issuecomment-1708065150
HI @markt-asf
I read codes in apache/coyote
I found
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
1001 - 1100 of 2012 matches
Mail list logo
