Skip to content

Commit

Permalink
Remove sending hash when registering; not used or required (#478)
Browse files Browse the repository at this point in the history
## Summary
<!-- Succinctly describe your change, providing context, what you've
changed, and why. -->

Removes `hash` from syncs to adhere to the spec.

## Checklist
<!-- Tick these items off as you progress. -->
<!-- If an item isn't applicable, ideally please strikeout the item by
wrapping it in "~~"" and suffix it with "N/A My reason for skipping
this." -->
<!-- e.g. "- [ ] ~~Added tests~~ N/A Only touches docs" -->

- [ ] ~~Added a [docs PR](https://github.com/inngest/website) that
references this PR~~ N/A
- [ ] ~~Added unit/integration tests~~ N/A
- [x] Added changesets if applicable

## Related

- #108
  • Loading branch information
jpwilliams committed Jan 31, 2024
1 parent b3a7b39 commit 9887ac4
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 7 deletions.
5 changes: 5 additions & 0 deletions .changeset/lazy-lies-film.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"inngest": patch
---

Remove sending `hash` when syncing
2 changes: 0 additions & 2 deletions packages/inngest/src/components/InngestCommHandler.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1058,8 +1058,6 @@ export class InngestCommHandler<
v: "0.1",
};

// Calculate the checksum of the body... without the checksum itself being included.
body.hash = sha256().update(canonicalize(body)).digest("hex");
return body;
}

Expand Down
5 changes: 0 additions & 5 deletions packages/inngest/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1088,11 +1088,6 @@ export interface RegisterRequest {
* The functions available at this particular handler.
*/
functions: FunctionConfig[];

/**
* The hash of the current commit used to track deploys
*/
hash?: string;
}

/**
Expand Down

0 comments on commit 9887ac4

Please sign in to comment.