You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Commit r32488 (bb77ad2) has added numerous entries into nselib/data/vhosts-default.lst, increasing its size five-fold.
Considering that the same additions were not made to nselib/data/vhosts-full.lst, it should be reviewed whether the update failed to account for existence of the second list and whether the bulk of these entries should not have been added into the full list instead, keeping the default list small for performance reasons. See commit 1441aea for additional information about the two lists.
One possible rule of thumb for the default list could be whether the entry likely represents a web interface. As an example, cms might be a good entry for the default list but dns2 might not.
P.S. All the missing entries have been added to the full list in r37929 (0d966d9).
The text was updated successfully, but these errors were encountered:
Commit r32488 (bb77ad2) has added numerous entries into
nselib/data/vhosts-default.lst
, increasing its size five-fold.Considering that the same additions were not made to
nselib/data/vhosts-full.lst
, it should be reviewed whether the update failed to account for existence of the second list and whether the bulk of these entries should not have been added into the full list instead, keeping the default list small for performance reasons. See commit 1441aea for additional information about the two lists.One possible rule of thumb for the default list could be whether the entry likely represents a web interface. As an example,
cms
might be a good entry for the default list butdns2
might not.P.S. All the missing entries have been added to the full list in r37929 (0d966d9).
The text was updated successfully, but these errors were encountered: