Skip to content

Implement Rum injection for servlet 3 #9102

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.

Sign up for GitHub

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

Merged
amarziali merged 13 commits into from
Jul 4, 2025
Merged

Implement Rum injection for servlet 3 #9102

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
257f167
feat(rum): Add initial config and API
PerfectSlayer Jul 3, 2025
5e734e3
feat(rum): Iterating on initial config and API
PerfectSlayer Jul 4, 2025
dde7164
fix(rum): Fix config print
PerfectSlayer Jul 4, 2025
bb22e1a
fix(rum): Remove content type support as only "text/html" is supported
PerfectSlayer Jul 4, 2025
1bf99a1
feat(rum): Add smoke tests
PerfectSlayer Jul 4, 2025
8814d86
feat(rum): Add smoke tests
PerfectSlayer Jul 4, 2025
a1f0a77
feat(rum): Add smoke tests
PerfectSlayer Jul 4, 2025
f941b4a
feat(rum): Add smoke tests
PerfectSlayer Jul 4, 2025
fe2006c
Add RUM injection for servlet 3
amarziali Jul 4, 2025
2ac167a
fix rum injection smoke test
amarziali Jul 4, 2025
c389b77
fix javadoc
amarziali Jul 4, 2025
01f6aae
Add benchmark
amarziali Jul 4, 2025
486ee57
avoid linkage issues with earlier servlet specs
amarziali Jul 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix javadoc
  • Loading branch information
@amarziali
amarziali committed Jul 4, 2025
commit c389b7767fdcc597dd17d25246da4babf0b8be3f
1 change: 1 addition & 0 deletions ...c/main/java/datadog/trace/bootstrap/instrumentation/buffer/InjectingPipeOutputStream.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ public class InjectingPipeOutputStream extends FilterOutputStream {
* @param downstream the delegate output stream
* @param marker the marker to find in the stream
* @param contentToInject the content to inject once before the marker if found.
* @param onContentInjected callback called when and if the content is injected.
*/
public InjectingPipeOutputStream(
final OutputStream downstream,
Expand Down