When installed on a Windows server, the absolute path option worked fine on 7.x-1.1, but after upgrading it no longer works.

I kept getting a file not found error. After switching to a relative path it seems to have fixed the problem.

Comments

greggles’s picture

Priority: Normal » Critical

This seems like a release blocker. I haven't tested it, but it seems possible.

When you say "absolute path option" where is that an option?

ryantollefson’s picture

When using a key file and you fill out the field, so on Windows using a path like: "C:\apache\www\drupal\blah\keyfile.key" vs a relative path like "../../blah/keyfile.key"

Hopefully that makes sense.

After a bit more testing, I was still having problems with the relative path - though I was not always getting an error. For now I switched to the "Drupal encrypt_drupal_variable_key variable" and stored the key in settings.php instead of using the "File" option.

greggles’s picture

Do you have a mac/linux test environment to try this out? Those are the only places I've run this and the file path seems to work - I wonder if it's windows related?

ryantollefson’s picture

Unfortunately not at the moment. You are probably right that it is Windows related.

rlhawk’s picture

Does it work properly if you add a trailing slash to the directory path? Key retrieval is currently broken without one. See Set key file location correctly for more information.

greggles’s picture

Issue summary: View changes
Status: Active » Postponed (maintainer needs more info)

Moving this to needs more info since I couldn't replicate it.

gisle’s picture

This is tagged as "Critical", but I am unable to reproduce. More info has been requested, but none has been provided in four years.

Can anyone confirm that this still is a problem (and if so, provide precise instructions to reproduce).

If there are no replies confirming the bug in two weeks, I suggest to this issue close as "cannot reproduce".

greggles’s picture

Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

Yep, agreed.