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

Errors in user-feedback-script #9065

Closed
thoschi opened this issue Apr 13, 2020 · 6 comments
Closed

Errors in user-feedback-script #9065

thoschi opened this issue Apr 13, 2020 · 6 comments

Comments

@thoschi
Copy link

@thoschi thoschi commented Apr 13, 2020

Not sure, if this is the correct place to post:

There seems to be at least two errors in the script mentioned at
https://docs.bigbluebutton.org/2.2/customize.html#collect-feedback-from-the-users

There seems to be a HERE missing before the second cat (about line 19).

Also the variable $PROTOCOL is used but not defined. The address created in /usr/share/meteor/bundle/programs/server/assets/app/config/settings.yml is therefore garbled and there is no output in html5-client.log.

Changing the address manually (e.g. https://...) solves the problem. So the script should do sth. like

PROTOCOL=$(cat` /usr/share/bbb-web/WEB-INF/classes/bigbluebutton.properties | grep -v '#' | grep '^bigbluebutton.web.serverURL' | sed 's/.*\(http[s]*\).*/\1/')

Enjoying BBB very much! Best regards, Thomas

@mleu77
Copy link

@mleu77 mleu77 commented Apr 15, 2020

Thank you for that hint.
After running the script (original and modified) I can't start nginx anymore.
Error message
nginx: [emerg] unknown log format "postdata" in /etc/bigbluebutton/nginx/html5-client-log.nginx

Do you have a solution for that problem?
Regards

@thoschi
Copy link
Author

@thoschi thoschi commented Apr 16, 2020

Yes this is exactely what happens, if you just execute the script as it is shown in the documentation.

First, edit /etc/bigbluebutton/nginx/html5-client-log.nginx. It should just contain:

location /html5log {
        access_log /var/log/nginx/html5-client.log postdata;
        echo_read_request_body;
}

remove anything else (there might be a block starting with cat).

Then check /etc/nginx/conf.d/html5-client-log.conf to have this line:

log_format postdata '\$remote_addr [\$time_iso8601] \$request_body';

Afterwards restart the nginx service.

Best regards
Thomas

@mleu77
Copy link

@mleu77 mleu77 commented Apr 16, 2020

Thank you very much - now it's working.
I think the tail - command for monitoring should also be improved.
tail -f /var/log/nginx/html5-client.log | sed -u 's/\\x22/"/g' | sed -u 's/\\x5C//g'

The command should reduce information e.g. just to "timestamp" and the "feedback": {rating.....}

Best regards
Martin

@matiasilva
Copy link
Collaborator

@matiasilva matiasilva commented Jun 18, 2020

I've submitted a pull request that fixes the above changes.

@matiasilva
Copy link
Collaborator

@matiasilva matiasilva commented Jun 18, 2020

PR now merged. Closing.

@Helyux
Copy link

@Helyux Helyux commented Nov 13, 2020

This Script i made may be helpful to some of you trying to read the logs.

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

No branches or pull requests

4 participants