Satyr can't distinguish between read errors and parse errors in config.yml #6

Closed
opened 2 years ago by gdh · 4 comments
gdh commented 2 years ago

I discovered that doing npm as root is something of an anti-pattern, and successfully ran npm install as the ubuntu user.

I executed the MariaDB script and renamed the generated config yml to config/config.yml, and yet...

ubuntu@ip-172-31-19-107:~/satyr$ npm run start

> satyr@0.10.1 start /home/ubuntu/satyr
> ts-node src/index.ts

No config file found. Exiting.
ubuntu@ip-172-31-19-107:~/satyr$ ls -l config/config.yml 
-rw-rw-r-- 1 ubuntu ubuntu 1330 Oct 20 22:40 config/config.yml

Suggestions welcome!

I discovered that doing `npm` as root is something of an anti-pattern, and successfully ran `npm install` as the `ubuntu` user. I executed the MariaDB script and renamed the generated config yml to config/config.yml, and yet... ``` ubuntu@ip-172-31-19-107:~/satyr$ npm run start > satyr@0.10.1 start /home/ubuntu/satyr > ts-node src/index.ts No config file found. Exiting. ubuntu@ip-172-31-19-107:~/satyr$ ls -l config/config.yml -rw-rw-r-- 1 ubuntu ubuntu 1330 Oct 20 22:40 config/config.yml ``` Suggestions welcome!
Owner

There's an error in the generated config syntax. I've pushed a fix.

To fix yours, go to the last line of config.yml and add a closing ' to the end of ingest: 'rtmp://live-ord02.twitch.tv/app/

There's an error in the generated config syntax. I've pushed a fix. To fix yours, go to the last line of config.yml and add a closing ' to the end of `ingest: 'rtmp://live-ord02.twitch.tv/app/`
Owner

It does raise the issue that satyr can't distinguish between syntax errors and file read errors though. I'm going to leave this open and change title.

It does raise the issue that satyr can't distinguish between syntax errors and file read errors though. I'm going to leave this open and change title.
knotteye changed title from No config file found to Satyr can't distinguish between read errors and parse errors in config.yml 2 years ago
gdh commented 2 years ago
Poster

Super - that fixed it, thanks :)

Super - that fixed it, thanks :)
Owner

Fixed in #13

Fixed in #13
knotteye closed this issue 2 years ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.