Project

General

Profile

Actions

User story #2211

closed

PT distributePolicy : Check if the WebDAV password is OK

Added by Matthieu CERDA almost 13 years ago. Updated over 9 years ago.

Status:
Released
Priority:
2
Assignee:
Matthieu CERDA
Category:
Techniques
Target version:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

PT distributePolicy : Check if the WebDAV password is OK


Related issues 2 (0 open2 closed)

Related to Rudder - User story #2248: WebDAV password for sending inventories should be a system variableRejectedActions
Blocked by Rudder - User story #856: Make promises in order to assure that Rudder services are still runningReleasedMatthieu CERDA2011-02-16Actions
Actions #1

Updated by Matthieu CERDA almost 13 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100

Applied in changeset commit:680777ab52fba83aa4594d67c0f2bba22dac0594.

Actions #2

Updated by Jonathan CLARKE almost 13 years ago

  • Status changed from Pending technical review to 2
  • % Done changed from 100 to 90
This looks OK, but for 1 thing:
  • I think we should avoid using permissions 644 for files containing passwords. Can you test with 600 or 640?
Actions #3

Updated by Matthieu CERDA almost 13 years ago

  • Status changed from 2 to Discussion

As I thought, it breaks apache. I propose that we use correct permissions in this case, namely either "www-data:www-data mode 600" or "root:www-data mode 640" (with the group adjusted to accomodate the Debian/SuSE flavors)

Actions #4

Updated by Matthieu CERDA almost 13 years ago

  • Status changed from Discussion to In progress

OK. Let's go with root:www-data and mode 640 (wwwrun for SLES).

Actions #5

Updated by Matthieu CERDA almost 13 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 90 to 100

Applied in changeset commit:7629f780b6ce03714f3d8137ddb2d396bd9e8dce.

Actions #6

Updated by Jonathan CLARKE almost 13 years ago

  • Status changed from Pending technical review to 10

This looks fine. It seems pretty future compatible too, with "!debian" as a default value. Good work.

Actions #7

Updated by Jonathan CLARKE almost 13 years ago

  • Status changed from 10 to Released
Actions #8

Updated by Benoît PECCATTE over 9 years ago

  • Project changed from 24 to Rudder
  • Category changed from Policy Templates to Techniques
Actions

Also available in: Atom PDF