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.

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

fix: validate agent name #271

Merged
merged 2 commits into from
Dec 8, 2022
Merged

fix: validate agent name #271

merged 2 commits into from
Dec 8, 2022

Conversation

alanshaw
Copy link
Member

@alanshaw alanshaw commented Dec 8, 2022

Cannot put undefined in UCANs

/** @type {import('./types').SpaceMeta} */
const meta = { isRegistered: false }
// eslint-disable-next-line eqeqeq
if (name != undefined) {
Copy link
Member Author

Choose a reason for hiding this comment

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

prettier has changed this from null to undefined but I think it has the same effect.

@alanshaw
Copy link
Member Author

alanshaw commented Dec 8, 2022

Going to merge...

@alanshaw alanshaw merged commit cdccbd3 into main Dec 8, 2022
@alanshaw alanshaw deleted the fix/validate-agent-name branch December 8, 2022 13:10
alanshaw pushed a commit that referenced this pull request Dec 8, 2022
🤖 I have created a release *beep* *boop*
---


##
[9.0.1](access-v9.0.0...access-v9.0.1)
(2022-12-08)


### Bug Fixes

* validate agent name
([#271](#271))
([cdccbd3](cdccbd3))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
alanshaw pushed a commit that referenced this pull request Dec 8, 2022
Cannot put `undefined` in UCANs
gobengo pushed a commit that referenced this pull request Apr 11, 2023
Cannot put `undefined` in UCANs
gobengo pushed a commit that referenced this pull request Apr 11, 2023
🤖 I have created a release *beep* *boop*
---


##
[9.0.1](access-v9.0.0...access-v9.0.1)
(2022-12-08)


### Bug Fixes

* validate agent name
([#271](#271))
([5a1fdb2](5a1fdb2))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
gobengo pushed a commit that referenced this pull request Apr 11, 2023
Cannot put `undefined` in UCANs
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.

1 participant