Skip to content

feat: add teleportr API server#2251

Merged
mslipper merged 5 commits intoethereum-optimism:developfrom
cfromknecht:teleportr-api-server
Mar 11, 2022
Merged

feat: add teleportr API server#2251
mslipper merged 5 commits intoethereum-optimism:developfrom
cfromknecht:teleportr-api-server

Conversation

@cfromknecht
Copy link
Copy Markdown
Contributor

@cfromknecht cfromknecht commented Mar 3, 2022

Description
This PR adds the teleport API server that will be used to facilitate requests for the Gateway and track deposits/disbursements. Posting this sketch to get early feedback on the desired API.

Metadata

  • Fixes ENG-1933

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 3, 2022

🦋 Changeset detected

Latest commit: f4f3054

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/teleportr Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@cfromknecht cfromknecht force-pushed the teleportr-api-server branch 3 times, most recently from cccbd96 to 118ce76 Compare March 3, 2022 18:00
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 3, 2022

Codecov Report

Merging #2251 (d1bf42a) into develop (e71307f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2251   +/-   ##
========================================
  Coverage    80.08%   80.08%           
========================================
  Files           77       77           
  Lines         2460     2460           
  Branches       450      450           
========================================
  Hits          1970     1970           
  Misses         490      490           
Flag Coverage Δ
contracts 99.29% <ø> (ø)
core-utils 86.77% <ø> (ø)
data-transport-layer 49.72% <ø> (ø)
sdk 55.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e71307f...d1bf42a. Read the comment docs.

@cfromknecht cfromknecht force-pushed the teleportr-api-server branch 3 times, most recently from 64955ea to 26f4f48 Compare March 3, 2022 22:58
Copy link
Copy Markdown
Contributor

@mslipper mslipper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should also add some HTTP request metrics as well - e.g., # of requests per path, # of HTTP errors, etc.

@cfromknecht cfromknecht force-pushed the teleportr-api-server branch from 26f4f48 to d1bf42a Compare March 10, 2022 19:27
@cfromknecht cfromknecht force-pushed the teleportr-api-server branch from d1bf42a to f4f3054 Compare March 11, 2022 14:08
@mslipper mslipper merged commit 58ce71f into ethereum-optimism:develop Mar 11, 2022
@cfromknecht cfromknecht deleted the teleportr-api-server branch March 14, 2022 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants