Relative Path for the htpasswd file does not work on linux

[lazaridis_com]

I've installed the account manager plugin and it works fine.

​http://dev.lazaridis.com/base/changeset/111

One problem occours:

In the local installation (windows) the relative path within the *.ini works fine.

In the production server, the relative path does not work (giving an absolute path works nice).

[lazaridis_com]

configuration:

8 	[components]
9 	tracx.* = enabled
10 	acct_mgr.* = enabled
11 	acct_mgr.web_ui.RegistrationModule = disabled
12 	
13 	[account-manager]
14 	password_format = htpasswd
15 	password_file = ../trac.htpasswd

[lazaridis_com]

on the 0.11dev version (missing this from the "Trac Release" drop down in this trac installtion)

[lazaridis_com]

If i understand right, then at this location:

source:accountmanagerplugin/trunk/acct_mgr/htfile.py#rev=1518#L51​

a test should happen if the given 'filename' is existent.

If not, the filename should be constructed based on the current environment.

[Matt Good]

Replying to lazaridis_com:

If i understand right, then at this location:

source:accountmanagerplugin/trunk/acct_mgr/htfile.py#rev=1518#L51​

a test should happen if the given 'filename' is existent.

If not, the filename should be constructed based on the current environment.

No, the Option class is just used to define that as a setting for trac.ini.

Using a relative path at this time will be relative to the current working directory of the web server. I'll look into making this work relative to the Trac environment instead.

[anonymous]

Replying to mgood:

this code should do the work, possibly you can verify, I get some errors which seem to result out of the way you've implemented the inheritance:

    def __init__(self):
        if not os.path.isabs(self.filename_option):
            combined_name = os.path.join(self.env.path, self.filename_option)
            self.filename = os.path.normpath(combined_name)
            self.log.debug('account manager password-file: %s', self.filename)

[lazaridis_com]

Replying to anonymous:

that was me, shouldn't this site do something like "auto-login" (e.g. from authentication cache? )

[Matt Good]

(In [1581]) enable password_file setting to be relative to the project (fixes #903)

[anonymous]

the patch works technically, but there's a small cosmetic issue:

"_filename" appears on the Admin page instead of "filename".

[lazaridis_com]

Replying to anonymous: that was me again.

[Matt Good]

(In [2063]) make the filename option display nicely in admin page (fixes #903)