Skip to content

Commit f2dc09d

Browse files
authored
Merge pull request #2844 from cliffhall/fix/do-not-sample-on-subscribe
Do not initiate sampling on resource subscription.
2 parents 1d47d6b + 23b8279 commit f2dc09d

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

src/everything/everything.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -353,9 +353,6 @@ export const createServer = () => {
353353
server.setRequestHandler(SubscribeRequestSchema, async (request, extra) => {
354354
const { uri } = request.params;
355355
subscriptions.add(uri);
356-
357-
// Request sampling from client when someone subscribes
358-
await requestSampling("A new subscription was started", uri, undefined, extra.sendRequest);
359356
return {};
360357
});
361358

@@ -877,7 +874,7 @@ export const createServer = () => {
877874
}
878875

879876
const uri = `data:application/zip;base64,${await zip.generateAsync({ type: "base64" })}`;
880-
877+
881878
return {
882879
content: [
883880
{

0 commit comments

Comments
 (0)