Chenjp commented on PR #783:
URL: https://github.com/apache/tomcat/pull/783#issuecomment-2493135169
> Not interested in that one, sorry, Tomcat will implement what the Servlet
specification says.
@rmaucher till now, the 6.2 Spec is not officially published.
--
This is an automated
rmaucher closed pull request #783: revoke early hints support on HTTP 1.1
URL: https://github.com/apache/tomcat/pull/783
--
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 unsu
rmaucher commented on PR #783:
URL: https://github.com/apache/tomcat/pull/783#issuecomment-2493131301
Not interested in that one, sorry, Tomcat will implement what the Servlet
specification says.
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
Chenjp opened a new pull request, #784:
URL: https://github.com/apache/tomcat/pull/784
1. automatically flush when earlyHints called, to enable the possibility of
server-think-time (programmatic code in web app) between the Early Hints
headers and the final response. Speeding up page load
Chenjp commented on code in PR #784:
URL: https://github.com/apache/tomcat/pull/784#discussion_r1853754550
##
java/org/apache/coyote/http2/Stream.java:
##
@@ -604,10 +611,23 @@ final void writeAck() throws IOException {
final void writeEarlyHints() throws IOException {
Chenjp commented on PR #788:
URL: https://github.com/apache/tomcat/pull/788#issuecomment-2505613190
> I've removed the `.DS_Store` file and added it to `.gitignore`. The
presence of the other files is an indicator that something has gone wrong so
they should not be ignored. I can see an iss
markt-asf commented on PR #789:
URL: https://github.com/apache/tomcat/pull/789#issuecomment-2505615298
Yes, there should be an extra `)`. This has been fixed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
markt-asf closed pull request #789: fix typo
URL: https://github.com/apache/tomcat/pull/789
--
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
markt-asf commented on PR #767:
URL: https://github.com/apache/tomcat/pull/767#issuecomment-2505639468
No changes to merge here. Looking forward to additional PRs.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
markt-asf closed pull request #767: BZ69355: ExactRateLimiter
URL: https://github.com/apache/tomcat/pull/767
--
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-m
Chenjp commented on PR #767:
URL: https://github.com/apache/tomcat/pull/767#issuecomment-2505642213
another one next week.
--
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.
T
rmaucher commented on PR #788:
URL: https://github.com/apache/tomcat/pull/788#issuecomment-2505661595
Slack in #tomcat ?
--
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
markt-asf commented on PR #788:
URL: https://github.com/apache/tomcat/pull/788#issuecomment-2505594672
I've removed the `.DS_Store` file and added it to `.gitignore`. The presence
of the other files is an indicator that something has gone wrong so they should
not be ignored. I can see an is
markt-asf closed pull request #788: minor update on .gitignore
URL: https://github.com/apache/tomcat/pull/788
--
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-
markt-asf commented on PR #788:
URL: https://github.com/apache/tomcat/pull/788#issuecomment-2505633255
Apache projects prefer to communicate via public channels - typically the
mailing list - rather than IM or other non-public channels. What we lose in
terms of speed we more than gain in te
robertonr commented on PR #742:
URL: https://github.com/apache/tomcat/pull/742#issuecomment-2505865395
Hello proxies could be implemented dynamically using one of ASM libraries;
Reflection is very expensive and taking into consideration the huge number of
applications using Tomcat this chan
Chenjp opened a new pull request, #789:
URL: https://github.com/apache/tomcat/pull/789
minor typo fix
--
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
Chenjp opened a new pull request, #788:
URL: https://github.com/apache/tomcat/pull/788
ignore ds_store and ant test related outputs.
--
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 c
robertonr commented on PR #742:
URL: https://github.com/apache/tomcat/pull/742#issuecomment-2504365369
Hello, I think this change introduced an important performance penalty
comparing my tests using 10.1.26 vs 10.1.27
--
This is an automated message from the Apache Git Service.
To respond
markt-asf commented on PR #742:
URL: https://github.com/apache/tomcat/pull/742#issuecomment-2504405168
You should test the latest version (although I suspect you'll see the same
results).
Use of proxies rather than wrappers was a design decision for jdbc-pool so
it could handle chang
markt-asf commented on PR #723:
URL: https://github.com/apache/tomcat/pull/723#issuecomment-2506537080
I'm not seeing a clean way to differentiate between container set 400 status
and application set. Closing this draft PR. I'd be happy to consider a PR if a
clean solution can be found.
-
markt-asf closed pull request #723: Make status code configurable
URL: https://github.com/apache/tomcat/pull/723
--
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,
markt-asf merged PR #778:
URL: https://github.com/apache/tomcat/pull/778
--
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 #782:
URL: https://github.com/apache/tomcat/pull/782#issuecomment-2504291618
Tx. Applied a variation manually.
--
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 closed pull request #782: send 416 error to overlapping ranges request
URL: https://github.com/apache/tomcat/pull/782
--
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.
ebourg commented on code in PR #789:
URL: https://github.com/apache/tomcat/pull/789#discussion_r1861630238
##
java/org/apache/catalina/servlets/DefaultServlet.java:
##
@@ -1487,7 +1487,7 @@ protected Ranges parseRange(HttpServletRequest request,
HttpServletResponse resp
dependabot[bot] closed pull request #781: Bump
org.apache.tomcat:tomcat-catalina from 11.0.0-M22 to 11.0.0 in /modules/stuffed
URL: https://github.com/apache/tomcat/pull/781
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
dependabot[bot] commented on PR #781:
URL: https://github.com/apache/tomcat/pull/781#issuecomment-2484937892
Looks like org.apache.tomcat:tomcat-catalina is up-to-date now, so this is
no longer needed.
--
This is an automated message from the Apache Git Service.
To respond to the message,
gmshake commented on PR #782:
URL: https://github.com/apache/tomcat/pull/782#issuecomment-2485163715
Nice catch!
--
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 unsubscr
rmaucher commented on PR #780:
URL: https://github.com/apache/tomcat/pull/780#issuecomment-2483051390
Merged in main, 11 and 10.1. Test case added to 9.0.
--
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 t
rmaucher closed pull request #780: fix multipart/byteranges response body -
length attri of ContentRange
URL: https://github.com/apache/tomcat/pull/780
--
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 t
dependabot[bot] opened a new pull request, #781:
URL: https://github.com/apache/tomcat/pull/781
Bumps org.apache.tomcat:tomcat-catalina from 11.0.0-M22 to 11.0.0.
[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=
Chenjp opened a new pull request, #782:
URL: https://github.com/apache/tomcat/pull/782
request ranges validation - overlap detection added.
* invalid ranges - overlapping:
```
D:\git\github.com>curl http://localhost:55464/index.html -i -H "Range:
bytes=10-40,35-50"
HTTP/1.1 4
markt-asf closed pull request #777: bz69439 - fix: improper handling of
conflicting cache-control directives
URL: https://github.com/apache/tomcat/pull/777
--
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
markt-asf commented on PR #777:
URL: https://github.com/apache/tomcat/pull/777#issuecomment-2489364168
Thanks for the PR. I have applied a fix based on this PR.
Removing the cache-control header caching removes a deliberate performance
optimisation (and doesn't update the Javadoc to r
Chenjp opened a new pull request, #778:
URL: https://github.com/apache/tomcat/pull/778
per [rfc 9110 -
14.4](https://httpwg.org/specs/rfc9110.html#field.content-range)
- ContentRange - add isValid(), more length check.
- ContentRange - parse(): make sure the not-null return of pars
rmaucher commented on PR #778:
URL: https://github.com/apache/tomcat/pull/778#issuecomment-2479048290
Ok, that's correct, the end is also inclusive.
--
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 t
Chenjp commented on PR #778:
URL: https://github.com/apache/tomcat/pull/778#issuecomment-2478953548
> -1 Other that the lower case range, the shuffling around of the checks
introduces a much more serious bug (&& length > end).
A Content-Range field value is invalid if it contains a
[
rmaucher commented on PR #778:
URL: https://github.com/apache/tomcat/pull/778#issuecomment-2478347337
-1
Other that the lower case range, the shuffling around of the checks
introduces a much more serious bug (&& length > end).
--
This is an automated message from the Apache Git Service
markt-asf commented on PR #779:
URL: https://github.com/apache/tomcat/pull/779#issuecomment-2497763433
Thanks for the PR. I have applied a variation to 9.0.x, 10.1.x 11.0.x and
main. The main changes were:
- use a Servlet rather than a JSP
- refactored the code to make `application
markt-asf closed pull request #779: bz69442: Content type testcase
URL: https://github.com/apache/tomcat/pull/779
--
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
jorgediaz-lr opened a new pull request, #786:
URL: https://github.com/apache/tomcat/pull/786
https://bz.apache.org/bugzilla/show_bug.cgi?id=69477
I have detected that the 'allowedInternalProxies' param name in the Remote
IP Filter documentation here
https://tomcat.apache.org/tomcat-1
markt-asf merged PR #786:
URL: https://github.com/apache/tomcat/pull/786
--
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
kriths opened a new pull request, #787:
URL: https://github.com/apache/tomcat/pull/787
(no comment)
--
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:
jorgediaz-lr commented on PR #786:
URL: https://github.com/apache/tomcat/pull/786#issuecomment-2503706162
Thank you!
--
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 unsu
markt-asf commented on PR #775:
URL: https://github.com/apache/tomcat/pull/775#issuecomment-2503658944
Thanks for the PR. I have applied a modified version to 12.0.x, 11.0.x,
10.1.x and 9.0.x.
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
markt-asf closed pull request #775: RateLimitFilter - enhance exposeHeaders to
support "ietf rate limit h…
URL: https://github.com/apache/tomcat/pull/775
--
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 #787:
URL: https://github.com/apache/tomcat/pull/787#issuecomment-2503730093
Sorry. Looks like our work on this overlapped.
--
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 closed pull request #787: Fix unsetting of httpOnly and secure flag
URL: https://github.com/apache/tomcat/pull/787
--
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
Chenjp opened a new pull request, #792:
URL: https://github.com/apache/tomcat/pull/792
add CRLF after the last-chunk to build a valid chunked request body.
--
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
Chenjp commented on PR #767:
URL: https://github.com/apache/tomcat/pull/767#issuecomment-2480547768
> I'm in favour of adding both the headers and the exact rate limiter. The
PR just needs some refactoring and clean-up. Generally, several small changes
are easier to review than one big chan
Chenjp commented on code in PR #782:
URL: https://github.com/apache/tomcat/pull/782#discussion_r1857791659
##
java/org/apache/catalina/servlets/DefaultServlet.java:
##
@@ -1231,10 +1231,25 @@ private static boolean validate(ContentRange range) {
(range.getEnd()
Chenjp commented on code in PR #782:
URL: https://github.com/apache/tomcat/pull/782#discussion_r1857791659
##
java/org/apache/catalina/servlets/DefaultServlet.java:
##
@@ -1231,10 +1231,25 @@ private static boolean validate(ContentRange range) {
(range.getEnd()
markt-asf merged PR #792:
URL: https://github.com/apache/tomcat/pull/792
--
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 merged PR #791:
URL: https://github.com/apache/tomcat/pull/791
--
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 #790:
URL: https://github.com/apache/tomcat/pull/790#issuecomment-2514678141
Note: It is not required for a server to send `Accept-Ranges` for it to
process a request with `Range`.
--
This is an automated message from the Apache Git Service.
To respond to the mes
markt-asf merged PR #790:
URL: https://github.com/apache/tomcat/pull/790
--
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 #790:
URL: https://github.com/apache/tomcat/pull/790#issuecomment-2514634738
Needs some small changes but I'll do that after merging but before
back-porting.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
Chenjp commented on PR #790:
URL: https://github.com/apache/tomcat/pull/790#issuecomment-2515063984
> Note: It is not required for a server to send `Accept-Ranges` for it to
process a request with `Range`.
Does value of ```useAcceptRanges``` determine the server enable / disable
feat
markt-asf commented on PR #790:
URL: https://github.com/apache/tomcat/pull/790#issuecomment-2515073630
No.
--
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
markt-asf commented on PR #790:
URL: https://github.com/apache/tomcat/pull/790#issuecomment-2515112985
If they really want to do that - and I can't think of a valid reason they
would - they can write a Filter (or Valve) to remove the Range header.
--
This is an automated message from the
Chenjp commented on PR #790:
URL: https://github.com/apache/tomcat/pull/790#issuecomment-2515107111
If app developer decide to disable ***Range Requests*** feature, how to?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
Chenjp commented on PR #791:
URL: https://github.com/apache/tomcat/pull/791#issuecomment-2516674415
@markt-asf any issue?
--
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.
T
markt-asf commented on PR #791:
URL: https://github.com/apache/tomcat/pull/791#issuecomment-2516693987
Yes. This change has been reverted. It is not RFC 9110 compliant.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
markt-asf commented on PR #790:
URL: https://github.com/apache/tomcat/pull/790#issuecomment-2516715364
Look at the history of the code. You'll need to go back a long way. There is
definitely an argument for deprecating this option and removing it in Tomcat 12.
--
This is an automated mess
Chenjp commented on PR #791:
URL: https://github.com/apache/tomcat/pull/791#issuecomment-2516821173
> Yes. This change has been reverted. It is not RFC 9110 compliant.
It is not rfc required, remove this detector if reasonable cases endorse.
Rfc uses term MAY, not MUST: A server
rmaucher commented on PR #791:
URL: https://github.com/apache/tomcat/pull/791#issuecomment-2516870493
Konstantin has provided examples of requests out there with these kind of
ranges. Rejecting them provides no value to Tomcat, so the change was reverted.
--
This is an automated message f
markt-asf commented on PR #793:
URL: https://github.com/apache/tomcat/pull/793#issuecomment-2519959571
We are only interested in the headers. The body could be using any encoding
so whatever is chosen the body will be corrupted for some requests.
--
This is an automated message from the A
markt-asf closed pull request #793: Use UTF-8 for logging in Http11InputBuffer
URL: https://github.com/apache/tomcat/pull/793
--
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
markt-asf commented on PR #793:
URL: https://github.com/apache/tomcat/pull/793#issuecomment-2519884919
HTTP headers are expected to be ASCII, not UTF-8.
--
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
izeye commented on PR #793:
URL: https://github.com/apache/tomcat/pull/793#issuecomment-2519950305
@markt-asf Thanks for the feedback!
Did you mean to say that the log only contains headers?
With [this sample](https://github.com/izeye/hello-spring-boot/tree/web), if
I send the
Chenjp commented on PR #790:
URL: https://github.com/apache/tomcat/pull/790#issuecomment-2516346283
> No.
If server enable Range-Requests feature and the target resource also support
it, I think no good reason for the absence of ```Accept-Ranges: bytes```
response header.
--
This
Chenjp opened a new pull request, #794:
URL: https://github.com/apache/tomcat/pull/794
smaller PR from https://github.com/apache/tomcat/pull/767 .
If you need exact rate limiting and can accept a small decrease in
efficiency, ExactRateLimiter may be an alternative option.
--
This i
Chenjp commented on PR #791:
URL: https://github.com/apache/tomcat/pull/791#issuecomment-2522898421
Got it. If possible, send me those examples to study the scenario. thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
rw7 commented on PR #57:
URL:
https://github.com/apache/tomcat-jakartaee-migration/pull/57#issuecomment-2573056116
Would be nice if there was any feedback on this issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
Hakky54 opened a new pull request, #805:
URL: https://github.com/apache/tomcat/pull/805
This PR is a followup of the following earlier PR
https://github.com/apache/tomcat/pull/673 Although that pull request didn't get
merged, the code changes has been comitted to the main branch by the main
markt-asf commented on PR #60:
URL:
https://github.com/apache/tomcat-jakartaee-migration/pull/60#issuecomment-2575956789
This PR is attempting to address two separate issues. It is better to stick
to one issue per PR. I suggest moving the modification changes to a new PR (and
include an ex
ahinc711 opened a new issue, #62:
URL: https://github.com/apache/tomcat-jakartaee-migration/issues/62
I'm using this tool with the `EE` profile to try converting some
applications to Jakarta EE in the short term without uplifting all
dependencies. I'm seeing runtime errors like the followin
markt-asf commented on issue #62:
URL:
https://github.com/apache/tomcat-jakartaee-migration/issues/62#issuecomment-2577142902
Hmm. The migration tool is designed on the basis that it "just" needs to
rename some packages from `javax. ...` to `jakarta. ...`. What you are trying
to do looks a
dependabot[bot] opened a new pull request, #63:
URL: https://github.com/apache/tomcat-jakartaee-migration/pull/63
Bumps [org.apache.bcel:bcel](https://github.com/apache/commons-bcel) from
6.9.0 to 6.10.0.
Changelog
Sourced from https://github.com/apache/commons-bcel/blob/master/REL
dependabot[bot] opened a new pull request, #66:
URL: https://github.com/apache/tomcat-jakartaee-migration/pull/66
Bumps commons-io:commons-io from 2.16.1 to 2.18.0.
[
from 0.8.11 to 0.8.12.
Release notes
Sourced from https://github.com/jacoco/jacoco/releases";>o
dependabot[bot] opened a new pull request, #64:
URL: https://github.com/apache/tomcat-jakartaee-migration/pull/64
Bumps org.apache.ant:ant from 1.10.14 to 1.10.15.
[ {
markt-asf commented on code in PR #818:
URL: https://github.com/apache/tomcat/pull/818#discussion_r1939680064
##
java/org/apache/tomcat/util/http/RequestUtil.java:
##
@@ -123,19 +123,12 @@ public static boolean isSameOrigin(HttpServletRequest
request, String origin) {
koust6u commented on code in PR #818:
URL: https://github.com/apache/tomcat/pull/818#discussion_r1940452423
##
java/org/apache/tomcat/util/http/RequestUtil.java:
##
@@ -123,19 +123,12 @@ public static boolean isSameOrigin(HttpServletRequest
request, String origin) {
//
koust6u commented on code in PR #818:
URL: https://github.com/apache/tomcat/pull/818#discussion_r1940452423
##
java/org/apache/tomcat/util/http/RequestUtil.java:
##
@@ -123,19 +123,12 @@ public static boolean isSameOrigin(HttpServletRequest
request, String origin) {
//
Dmole commented on PR #814:
URL: https://github.com/apache/tomcat/pull/814#issuecomment-2631971994
I added it to both AccessLogValve and ExtendedAccessLogValve because it is
non-trivial for users to switch between them as the documentation does not
provide a mapping (EG: %v = ?), and the ex
Dmole commented on PR #815:
URL: https://github.com/apache/tomcat/pull/815#issuecomment-2631976321
Looks like it was not "intentional" after all
https://github.com/apache/tomcat/commit/904701a434325534b856b70f085dbe1dee05c43a
--
This is an automated message from the Apache Git Service.
markt-asf commented on PR #817:
URL: https://github.com/apache/tomcat/pull/817#issuecomment-2630498955
1 change per PR please.
Note that we try to avoid static imports if at all possible.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
markt-asf commented on PR #814:
URL: https://github.com/apache/tomcat/pull/814#issuecomment-2630509072
This is not the way to do this. Look at the ExtendedAccessLogValve. Look at
`x-H(...)` options.
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
funkman commented on code in PR #818:
URL: https://github.com/apache/tomcat/pull/818#discussion_r1939578798
##
test/org/apache/tomcat/util/http/TestRequestUtilSameOrigin.java:
##
@@ -42,6 +42,8 @@ public static Collection parameters() {
TesterRequest request2 = new Test
koust6u commented on code in PR #818:
URL: https://github.com/apache/tomcat/pull/818#discussion_r1939607782
##
test/org/apache/tomcat/util/http/TestRequestUtilSameOrigin.java:
##
@@ -42,6 +42,8 @@ public static Collection parameters() {
TesterRequest request2 = new Test
rmaucher commented on PR #815:
URL: https://github.com/apache/tomcat/pull/815#issuecomment-2631082908
> @rmaucher This is something worth mentioning in
https://tomcat.apache.org/migration-9.html#Tomcat_9.0.x_noteable_changes for
9.0.83 and similar releases in the other branches.
It's
koust6u closed pull request #817: Replace \r\n with SimpleHttpClient.CRLF in
Http2TestBase & Cleanup RequestUtil
URL: https://github.com/apache/tomcat/pull/817
--
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
koust6u opened a new pull request, #818:
URL: https://github.com/apache/tomcat/pull/818
(no comment)
--
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:
markt-asf commented on PR #816:
URL: https://github.com/apache/tomcat/pull/816#issuecomment-2630705976
`getResourceAsStream()` is documented to accept a name, not a URL. I'd like
to understand the use case better before making any changes. Can you provide
some sample code or a test case or
Dmole commented on PR #815:
URL: https://github.com/apache/tomcat/pull/815#issuecomment-2631122523
When editing the documentation please explain how requiring an empty
password, in a text file, in an unrelated attribute, qualifies as security, and
not just obscurity.
--
This is an automa
ChristopherSchultz commented on PR #815:
URL: https://github.com/apache/tomcat/pull/815#issuecomment-2631002992
@rmaucher This is something worth mentioning in
https://tomcat.apache.org/migration-9.html#Tomcat_9.0.x_noteable_changes for
9.0.83 and similar releases in the other branches.
-
1401 - 1500 of 2012 matches
Mail list logo
