Skip to content

chore(beak): bring in mojo packages #391

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

Jump to bottom
Merged
0xdeafcafe merged 3 commits into from
Jul 15, 2022
Merged

chore(beak): bring in mojo packages #391

0xdeafcafe merged 3 commits into from
Jul 15, 2022

Conversation

@0xdeafcafe
Copy link
Member

@0xdeafcafe 0xdeafcafe commented Jul 15, 2022

  • improved crpc/squawk
  • improved json-client
  • imported crpc
  • updated json-client to support timeouts

@0xdeafcafe 0xdeafcafe added enhancement New feature or request dependencies Pull requests that update a dependency file chore A real ugh labels Jul 15, 2022
github-advanced-security[bot]
github-advanced-security bot found potential problems Jul 15, 2022
View reviewed changes
packages/json-client/src/index.ts
};

export default function jsonClient(baseUrl: string, baseOptions?: RequestOptions) {
const safeBaseUrl = baseUrl.replace(/\/*$/, '/');

Check failure

Code scanning / CodeQL

Polynomial regular expression used on uncontrolled data

This [regular expression](1) that depends on [library input](2) may run slow on strings with many repetitions of '/'. This [regular expression](1) that depends on [library input](3) may run slow on strings with many repetitions of '/'.
@0xdeafcafe 0xdeafcafe merged commit e3a7cab into master Jul 15, 2022
@0xdeafcafe 0xdeafcafe deleted the chore/package-migration branch July 15, 2022 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

chore A real ugh dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

✨ Beak Roadmap
Status: Waiting Release

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@0xdeafcafe