Project

General

Profile

Bug #3294

Creating users/groups silently fails on read only filesystem

Added by Phil Sweeney over 7 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
User Manager / Privileges
Target version:
Start date:
10/26/2013
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Default
Affected Version:
2.1
Affected Architecture:
All

Description

Was setting up VPN and following various guides, which included the creation of a group and a user.

I did this, and it was all reported fine, however when I looked at the systemlog it reported an error due to filesystem being readonly.

To clean it up I had to remount as R/W, delete the user and groups, and start over.

Associated revisions

Revision b1e5a286 (diff)
Added by Renato Botelho over 7 years ago

Add conf_mount_rw calls on functions that changes user/groups. It fixes #3294

Revision fa757d14 (diff)
Added by Renato Botelho over 7 years ago

Add conf_mount_rw calls on functions that changes user/groups. It fixes #3294

Revision 23b5b16a (diff)
Added by Renato Botelho over 7 years ago

Revert "Add conf_mount_rw calls on functions that changes user/groups. It fixes #3294"

This reverts commit b1e5a286bb47d7e4a5b3d589cc27b557b3b13c41.

Revision 03e156ef (diff)
Added by Renato Botelho over 7 years ago

Revert "Add conf_mount_rw calls on functions that changes user/groups. It fixes #3294"

This reverts commit fa757d146c85261b7e90d226c1aecd38089d7e20.

History

#1 Updated by Renato Botelho over 7 years ago

  • Target version set to 2.1.1
  • Affected Architecture All added
  • Affected Architecture deleted (amd64)

#2 Updated by Renato Botelho over 7 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

#4 Updated by Ermal Lu├ži over 7 years ago

The commits are wrong in my opinion!

Why should backend code have to deal with mounting the disk.
I sept a lot of time before on having this cleaned up and now you are adding things back again :)

#5 Updated by Phil Sweeney over 7 years ago

When I reported this I was probably more expecting some error handling to be added so it actually failed properly and told you the error rather than just pretending it worked, but up to you guys :)

#8 Updated by Renato Botelho over 7 years ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF