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/conductor logs #125

Merged
merged 4 commits into from
Jun 16, 2022
Merged

Fix/conductor logs #125

merged 4 commits into from
Jun 16, 2022

Conversation

jost-s
Copy link
Contributor

@jost-s jost-s commented Jun 16, 2022

Fixed

  • Local conductor: subscribe to all messages to stderr.
  • Local conductor: log as info instead of as error.

fixes #124

@jost-s jost-s merged commit 59653f9 into develop Jun 16, 2022
@jost-s jost-s deleted the fix/conductor-logs branch June 16, 2022 18:21
jost-s added a commit that referenced this pull request Jun 21, 2022
* Update CHANGELOG.md

* Fix/conductor logs (#125)

fix(local-conductor): log conductor msgs as info

* feat(common): add optional timeout to zome call (#130)

* docs(readme): mention log level env var

* fix(local): log conductor startup only once

* chore(changelog): update and bump version

* build(deps): bump smallvec from 1.6.0 to 1.8.0 in /crates/trycp_server (#127)

Bumps [smallvec](https://github.com/servo/rust-smallvec) from 1.6.0 to 1.8.0.
- [Release notes](https://github.com/servo/rust-smallvec/releases)
- [Commits](servo/rust-smallvec@v1.6.0...v1.8.0)

---
updated-dependencies:
- dependency-name: smallvec
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
jost-s added a commit that referenced this pull request Jun 22, 2022
* Update CHANGELOG.md

* Fix/conductor logs (#125)

fix(local-conductor): log conductor msgs as info

* feat(common): add optional timeout to zome call (#130)

* docs(readme): mention log level env var

* fix(local): log conductor startup only once

* chore(changelog): update and bump version

* build(deps): bump smallvec from 1.6.0 to 1.8.0 in /crates/trycp_server (#127)

Bumps [smallvec](https://github.com/servo/rust-smallvec) from 1.6.0 to 1.8.0.
- [Release notes](https://github.com/servo/rust-smallvec/releases)
- [Commits](servo/rust-smallvec@v1.6.0...v1.8.0)

---
updated-dependencies:
- dependency-name: smallvec
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(local): log conductor output at debug level (#132)

* chore(changelog): update and bump version

* fix(local): print log as string

* ci(test): only run on PR

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
jost-s added a commit that referenced this pull request Jul 2, 2022
* Update CHANGELOG.md

* Fix/conductor logs (#125)

fix(local-conductor): log conductor msgs as info

* feat(common): add optional timeout to zome call (#130)

* docs(readme): mention log level env var

* fix(local): log conductor startup only once

* chore(changelog): update and bump version

* build(deps): bump smallvec from 1.6.0 to 1.8.0 in /crates/trycp_server (#127)

Bumps [smallvec](https://github.com/servo/rust-smallvec) from 1.6.0 to 1.8.0.
- [Release notes](https://github.com/servo/rust-smallvec/releases)
- [Commits](servo/rust-smallvec@v1.6.0...v1.8.0)

---
updated-dependencies:
- dependency-name: smallvec
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(local): log conductor output at debug level (#132)

* chore(changelog): update and bump version

* fix(local): print log as string

* ci(test): only run on PR

* add dna properties on install; dedup

* add dna properties on install - trycp; add to `addPlayerWithHapp`

* make option

* add missing field

* update docs, manage non-array case

* clean up

* refactor(common): mark register dna req opts internal

* chore(build): update changelog & bump version

* feat(common): add dna properties on install (#135)

* add dna properties on install; dedup

* add dna properties on install - trycp; add to `addPlayerWithHapp`

* make option

* add missing field

* update docs, manage non-array case

* clean up

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: jetttech <ldjetton21@gmail.com>
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.

unless making this .on instead of .once all the conductor logs are swallowed here
1 participant