Enhancement #2927

Notify user if event fails

Added by Davide Principi almost 7 years ago. Updated over 6 years ago.

Status:CLOSEDStart date:
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:<multiple packages>
Target version:v6.5
Resolution: NEEDINFO:No

Description

At the moment, Nethgui reports failed events only for detached tasks (those started with &). Synchronous tasks, such as user-create never report a failure condition.


Related issues

Related to NethServer 6 - Bug #2944: User-create event fails if user name is "service" CLOSED
Copied to Nethgui - Enhancement #2949: Notify user if event fails CLOSED

Associated revisions

Revision 36ce1e7d
Added by Davide Principi almost 7 years ago

SystemTasks: handle special socket open errors. Refs #2927

- Removed dump cleaning up code: never called
- Unlink socket, if stale

Revision 13de0bc8
Added by Davide Principi over 6 years ago

Call PHP exec() to invoke db command. Refs #2927

The Process class is deprecated.

Revision 1db96240
Added by Davide Principi over 6 years ago

Tweaked Tracker behaviour. Refs #2927

- Don't flash a notification when a detached task is started
- Require user authentication
- Throw a 404 error on socket exceptions

Revision f98716d2
Added by Davide Principi over 6 years ago

Nethgui 1.6.1 provides automatic reload of the url of the last POST requested. Refs #2927

Revision 225e423f
Added by Davide Principi over 6 years ago

Show failure notifications for synchronous events. Refs #2927

  • NethPlatform: added setDetachedProcessCondition() method
    (experimental).
  • Process: completed migration of features to NethPlatform; it is only
    an interface adapter to Symfony/Process

Revision 8eae1f73
Added by Davide Principi over 6 years ago

Require nethserver-lib > 2.1.1-1 for newer ptrack release. Refs #2927

Revision 6f8f563a
Added by Davide Principi over 6 years ago

Start detached hostname-modify event. Refs #2927

The event restarts httpd-admin and must run detached, to report errors
correctly.

Revision 38f70738
Added by Davide Principi over 6 years ago

Fixed setDetachedProcessCondition() to work with CompositeController. Refs #2927

Unfortunately, CompositeController does not honor nextPath()..

Revision d7e506d5
Added by Davide Principi over 6 years ago

FQDN page: updated warning message. Refs #2927

The connection is lost during the event due to new SSL certificates.

The new warning message explains to reload the page manually, only if
no custom certs are installed.

Revision f08aa5f8
Added by Davide Principi over 6 years ago

Remove ptrack *.dump files older than one week. Refs #2927

Revision 3aa551be
Added by Davide Principi over 6 years ago

Bump nethgui 38f70738b9cfe2c51368e6b95a52d2c9c555788b. Refs #2927

Revision 552336f4
Added by Davide Principi over 6 years ago

Require recent nethserver-lib, to show event errors. Refs #2927

Requires: nethserver-lib > 2.1.1-1

Revision a06dff14
Added by Davide Principi over 6 years ago

Start detached hostname-modify event. Refs #2927

The event restarts httpd-admin and must run detached, to report errors
correctly.

Revision 660c2800
Added by Davide Principi over 6 years ago

FQDN page: updated warning message. Refs #2927

The connection is lost during the event due to new SSL certificates.

The new warning message explains to reload the page manually, only if
no custom certs are installed.

History

#1 Updated by Davide Principi almost 7 years ago

  • Status changed from TRIAGED to ON_DEV
  • Assignee set to Davide Principi
  • % Done changed from 20 to 30

#2 Updated by Davide Principi almost 7 years ago

  • Related to Bug #2944: User-create event fails if user name is "service" added

#3 Updated by Davide Principi over 6 years ago

  • Project changed from NethServer 6 to Nethgui
  • Category deleted (<multiple packages>)
  • Target version changed from v6.5 to v1.6.1

#4 Updated by Davide Principi over 6 years ago

#5 Updated by Davide Principi over 6 years ago

  • Project changed from Nethgui to NethServer 6
  • Category set to <multiple packages>
  • Status changed from ON_DEV to MODIFIED
  • Target version changed from v1.6.1 to v6.5
  • % Done changed from 30 to 60

Test case

The UI must now show notifications of failed processes. Three types of processes must be tested:
  • asynchronous exec() calls, such as those invoked by PackageManager page
  • asynchronous signalEvent(), such as those in DNS page
  • synchronous signalEvent() calls, such as UserProfile one

QA note
Tracked also downstream package nethserver-register
https://dev.nethesis.it/issues/3243

#6 Updated by Davide Principi over 6 years ago

  • Status changed from MODIFIED to ON_QA
  • Assignee deleted (Davide Principi)
  • % Done changed from 60 to 70

In nethserver-testing:
nethserver-base-2.5.2-1.0git03cc8196.ns6.noarch.rpm
nethserver-httpd-admin-1.3.3-1.2git8eae1f7.ns6.noarch.rpm
nethserver-lib-2.1.1-1.0git647ddd7f.ns6.noarch.rpm
nethserver-firewall-base-2.2.0-1.1gitf98716d.ns6.noarch.rpm

#7 Updated by Davide Principi over 6 years ago

  • Status changed from ON_QA to TRIAGED
  • % Done changed from 70 to 20

The default callback URL does not fit the TableController case

#8 Updated by Davide Principi over 6 years ago

  • Status changed from TRIAGED to ON_DEV
  • Assignee set to Davide Principi
  • % Done changed from 20 to 30

#9 Updated by Davide Principi over 6 years ago

  • Status changed from ON_DEV to MODIFIED
  • Assignee deleted (Davide Principi)
  • % Done changed from 30 to 60

MODIFIED

Test case

see previous test case, moreover

  • added a cron job in nethserver-lib that removes *.dump files in /var/spool/ptrack
  • run hostname-modify in nethserver-base detached
  • the UI warning in FQDN page changes if SSL certs are set in pki DB key.

#10 Updated by Davide Principi over 6 years ago

  • Status changed from MODIFIED to ON_QA
  • % Done changed from 60 to 70

In nethserver-testing:
nethserver-lib-2.1.1-2.0gitf08aa5f8.ns6.noarch.rpm
nethserver-httpd-admin-1.3.3-1.3gita80cc74.ns6.noarch.rpm
nethserver-httpd-admin-1.3.3-1.4git3aa551b.ns6.noarch.rpm
nethserver-base-2.5.2-6.0gitd7e506d5.ns6.noarch.rpm
nethserver-firewall-base-2.2.0-1.1gitf98716d.ns6.noarch.rpm

Release note

The UI modules in firewall-base and base packages depends on newer httpd-admin version. That package must be updated manually!

#11 Updated by Giacomo Sanchietti over 6 years ago

  • Assignee set to Giacomo Sanchietti

#12 Updated by Giacomo Sanchietti over 6 years ago

  • Status changed from ON_QA to VERIFIED
  • Assignee deleted (Giacomo Sanchietti)
  • % Done changed from 70 to 90

VERIFIED

After changing the FQDN, the tracker will block because the browser doesn't accept the changed certificate.
Maybe we can use a wildcard certificate just for httpd-admin to avoid this problem.

#13 Updated by Giacomo Sanchietti over 6 years ago

  • Status changed from VERIFIED to CLOSED
  • % Done changed from 90 to 100
Released in nethserver-updates:
  • nethserver-lib-2.1.2-1.ns6.noarch.rpm
  • nethserver-httpd-admin-1.3.4-1.ns6.noarch.rpm
  • nethserver-firewall-base-2.2.1-1.ns6.noarch.rpm
  • nethserver-base-2.5.3-1.ns6.noarch.rpm

Also available in: Atom PDF