Skip to content
Snippets Groups Projects

Resolve #3437319 "Add validation constraints to update.settings"

Open Resolve #3437319 "Add validation constraints to update.settings"
1 unresolved thread
1 unresolved thread

Closes #3437319

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Wim Leers
  • Wim Leers
  • Srishti Bankar added 1 commit

    added 1 commit

    • fce132fb - remove nullable from notification.emails in favor of empty array

    Compare with previous version

  • Srishti Bankar resolved all threads

    resolved all threads

  • Wim Leers
    Wim Leers @wimleers started a thread on commit fce132fb
  • 37 37 timeout:
    38 38 type: integer
    39 39 label: 'Timeout in seconds'
    40 constraints:
    41 Range:
    42 min: 0
    • Comment on lines +40 to +42

      :thinking: I don't think timeout being set to 0 would result in a well-working application: if I read \Drupal\update\UpdateProcessor::fetchData() correctly, it might result in no update data being fetched, or perhaps a single item. A single item would be fine, zero would not be.

      I think 1 would probably be a better minimum, but there's no real harm caused by this — the status report would still end up complaining about updates not having been checked in a long time. Which could also happen if the timeout was set to 1 and d.o is slow and/or the server hosting the Drupal application has a slow internet connection.

      So … not commit-blocking IMHO.

    • Please register or sign in to reply
    Please register or sign in to reply
    Loading