generate-names.pl --compress does not produces .htaccess file #541

Closed
carrere opened this Issue Nov 28, 2014 · 3 comments

Comments

Projects
None yet
2 participants
@carrere

carrere commented Nov 28, 2014

I think there's a bug with the compression on "names" directory.

Using the command (tested on last Jbrowse release 1.11.5):

./bin/generate-names.pl --compress --out /path/to/data

All files are correctly compressed:

ls -l /path/to/data/names/ | head -5
-rw-r--r-- 1 root root 2200 Nov 28 11:04 00.jsonz
-rw-r--r-- 1 root root 1893 Nov 28 11:04 01.jsonz
-rw-r--r-- 1 root root 2663 Nov 28 11:04 02.jsonz
-rw-r--r-- 1 root root 2322 Nov 28 11:04 03.jsonz

But the script does not produce the required .htaccess file (as the flatfile-to-json.pl or prepare-refseq.pl do) that contains decompression directives:

mod_gzip_item_exclude "(.txtz|.jsonz)$" SetEnvIf Request_URI "(.txtz|.jsonz)$" no-gzip dont-vary Header onsuccess set Content-Encoding gzip

As a result, web browser is unable to propose autocompletion.
Error reading from name store.

So we cannot compress "names" what could become problematic for big datasets.

@cmdcolin

This comment has been minimized.

Show comment
Hide comment
@cmdcolin

cmdcolin Dec 1, 2014

Contributor

Thanks for the report! I added a patch that I think fixes it by creating a similar htaccess file. If you want to test it then you can look at the new source code for the IndexNames.pm and see if that helps.

Contributor

cmdcolin commented Dec 1, 2014

Thanks for the report! I added a patch that I think fixes it by creating a similar htaccess file. If you want to test it then you can look at the new source code for the IndexNames.pm and see if that helps.

cmdcolin added a commit that referenced this issue Dec 2, 2014

@carrere

This comment has been minimized.

Show comment
Hide comment
@carrere

carrere Dec 3, 2014

Thanks,

I retrieved the IndexNames.pm and that fixes perfectly the problem.

Sébastien

On 01/12/2014 20:48, Colin Diesh wrote:

Thanks for the report! I added a patch that I think fixes it by
creating a similar htaccess file. If you want to test it then you can
look at the new source code for the IndexNames.pm and see if that helps.


Reply to this email directly or view it on GitHub
#541 (comment).

carrere commented Dec 3, 2014

Thanks,

I retrieved the IndexNames.pm and that fixes perfectly the problem.

Sébastien

On 01/12/2014 20:48, Colin Diesh wrote:

Thanks for the report! I added a patch that I think fixes it by
creating a similar htaccess file. If you want to test it then you can
look at the new source code for the IndexNames.pm and see if that helps.


Reply to this email directly or view it on GitHub
#541 (comment).

@cmdcolin

This comment has been minimized.

Show comment
Hide comment
@cmdcolin

cmdcolin Dec 3, 2014

Contributor

Super :) closed for now

Contributor

cmdcolin commented Dec 3, 2014

Super :) closed for now

@cmdcolin cmdcolin closed this Dec 3, 2014

@cmdcolin cmdcolin added this to the 1.11.6 milestone Jan 23, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment