Skip to content
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

feat: use Blob in sendBeacon to add application/json type #2336

Merged
merged 14 commits into from
Aug 10, 2021
Merged

feat: use Blob in sendBeacon to add application/json type #2336

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
6feaea8
fix(opentelemetry-exporter-collector): issue#2321 use Blob for method…
jufab Jul 7, 2021
3cc3c90
fix: test unit browser error
jufab Jul 8, 2021
9d7a4f7
Merge branch 'main' into fix-issue-2321
jufab Jul 22, 2021
9cf813c
feat: add contentType configuration for beacon
jufab Jul 26, 2021
1c68c75
fix: some lint
jufab Jul 26, 2021
b891b21
Merge branch 'open-telemetry:main' into fix-issue-2321
jufab Jul 26, 2021
c5fdb2a
Merge branch 'open-telemetry:main' into fix-issue-2321
jufab Jul 27, 2021
9218835
fix: use BlobPropertyBag for Blob
jufab Jul 27, 2021
e07167b
Merge branch 'open-telemetry:main' into fix-issue-2321
jufab Aug 1, 2021
e9207d1
fix: assertion name
jufab Aug 1, 2021
17e27ae
fix: blob on public sendWithBeacon only
jufab Aug 3, 2021
120cb87
Merge branch 'main' into fix-issue-2321
dyladan Aug 9, 2021
fdb7a4b
Merge branch 'main' into fix-issue-2321
dyladan Aug 10, 2021
5ccc02e
Style nit
dyladan Aug 10, 2021
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
2 changes: 1 addition & 1 deletion packages/opentelemetry-exporter-collector/src/platform/browser/util.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ export function sendWithBeacon(
onSuccess: () => void,
onError: (error: collectorTypes.CollectorExporterError) => void
) {
if (navigator.sendBeacon(url, body)) {
if (navigator.sendBeacon(url, new Blob([body], { type: "application/json" }))) {
vmarchaud marked this conversation as resolved.
Show resolved Hide resolved
diag.debug('sendBeacon - can send', body);
onSuccess();
} else {
Expand Down
5 changes: 3 additions & 2 deletions packages/opentelemetry-exporter-collector/test/browser/CollectorMetricExporter.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,10 +93,11 @@ describe('CollectorMetricExporter - web', () => {
it('should successfully send metrics using sendBeacon', done => {
collectorExporter.export(metrics, () => {});

setTimeout(() => {
setTimeout(async () => {
const args = stubBeacon.args[0];
const url = args[0];
const body = args[1];
const blob:Blob = args[1];
dyladan marked this conversation as resolved.
Show resolved Hide resolved
const body = await blob.text();
const json = JSON.parse(
body
) as collectorTypes.opentelemetryProto.collector.metrics.v1.ExportMetricsServiceRequest;
Expand Down
5 changes: 3 additions & 2 deletions packages/opentelemetry-exporter-collector/test/browser/CollectorTraceExporter.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,10 +69,11 @@ describe('CollectorTraceExporter - web', () => {
it('should successfully send the spans using sendBeacon', done => {
collectorTraceExporter.export(spans, () => {});

setTimeout(() => {
setTimeout(async () => {
const args = stubBeacon.args[0];
const url = args[0];
const body = args[1];
const blob:Blob = args[1];
const body = await blob.text();
const json = JSON.parse(
body
) as collectorTypes.opentelemetryProto.collector.trace.v1.ExportTraceServiceRequest;
Expand Down