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

Disable warning about zlib if compression is disabled globally #525

Closed
t2d opened this issue Sep 24, 2019 · 4 comments
Closed

Disable warning about zlib if compression is disabled globally #525

t2d opened this issue Sep 24, 2019 · 4 comments
Assignees
Labels

Comments

@t2d
Copy link

t2d commented Sep 24, 2019

First, thank you for making the great new release and improving compatibility with TorBrowser!

Steps to reproduce

  1. set compression = none in conf.php
  2. Visit privatebin with TorBrowser

What happens

It shows the warning " Your browser doesn't support WebAssembly, used for zlib compression. You can create uncompressed documents, but can't read compressed ones."

What should happen

Since compression is disabled globally, there is no need to warn about missing zlib compatibility.

Basic information

Browser: TorBrowser 8.5.5

PrivateBin version: 1.3.1

I can reproduce this issue on https://privatebin.net: Yes

@rugk rugk added the bug label Sep 24, 2019
@qianmengnet
Copy link
Contributor

add wasm to application / javascript in mime

@elrido
Copy link
Contributor

elrido commented Jan 8, 2020

Hi and sorry for the very late reply (in another decade, nonetheless).

I tried recreating this using a Firefox with disabled wasm support and a local test instance with the compression = "none" setting, but failed to do so. It should be addressed with commit 5471757, which was part of the 1.3.1 release. Would you have the opportunity to retest this on an instance with compression disabled using Tor & TorBrowser?

@t2d
Copy link
Author

t2d commented Jan 9, 2020

Thank you for looking into this. The problem is caused by weird parsing of the config file.

compression = none

shows the warning.

compression = "none"

disables it.

@elrido
Copy link
Contributor

elrido commented Jan 9, 2020

Argh, of course - PHP will parse none as False instead of a string. I'll clarify that in the conf.sample.php and the wiki page on configuration.

From the PHP manual on the function parse_ini_file():

Note: There are reserved words which must not be used as keys for ini files. These include: null, yes, no, true, false, on, off, none.

@elrido elrido self-assigned this Jan 9, 2020
elrido added a commit that referenced this issue Jan 11, 2020
@elrido elrido closed this as completed Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants