Bug 202655 - security/base: Correct snort URL in external_sig_link array
Summary: security/base: Correct snort URL in external_sig_link array
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Steve Wills
URL:
Keywords: needs-qa, patch
Depends on:
Blocks:
 
Reported: 2015-08-25 13:36 UTC by Jeremy Baggs
Modified: 2018-07-10 16:04 UTC (History)
2 users (show)

See Also:

Attachments
Patch to correct URL for snort SID lookup (726 bytes, patch)
2015-08-25 13:36 UTC, Jeremy Baggs 		no flags Details | Diff
Correct URL for snort SID lookup, correct URL in pkg-descr (1.75 KB, patch)
2016-10-13 09:30 UTC, VK 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Baggs 2015-08-25 13:36:49 UTC 
Created attachment 160348 [details]
Patch to correct URL for snort SID lookup

The URL for snort SID lookup has changed.
Comment 1 Raphael Kubo da Costa freebsd_committer freebsd_triage 2016-03-03 15:47:40 UTC 
Hi,

Will this apply only to the default (dist) file shipped by the port? Do users need to update their own configurations?
Comment 2 VK 2016-10-13 09:29:15 UTC 
I was about to close this for no feedback, but seeing the requested change, I'd rather ask if anyone familiar with Snort could pitch in? Is patching the URL sufficient, or something else needs to be done?

I'm attaching a complete svn patch per requested changes, but need more info per comment #1.
Comment 3 VK 2016-10-13 09:30:04 UTC 
Created attachment 175691 [details]
Correct URL for snort SID lookup, correct URL in pkg-descr
Comment 4 VK 2016-10-13 09:41:57 UTC 
Build tests passed: portlint (warnings from before), Poudriere amd64 11.0-p1, 10.3-p10 and 9.3-p47.

Keeping the needs-qa flag for requested feedback.
Comment 5 VK 2016-10-13 09:45:12 UTC 
Revert assignee, I'm sorry for the noise, I intended to close so the field remained...
Comment 6 Walter Schwarzenfeld 2018-02-08 17:41:13 UTC 
Is this still relevant?
Comment 7 commit-hook freebsd_committer freebsd_triage 2018-07-10 16:03:42 UTC 
A commit references this bug:

Author: swills
Date: Tue Jul 10 16:03:14 UTC 2018
New revision: 474378
URL: https://svnweb.freebsd.org/changeset/ports/474378

Log:
  security/base: Correct snort URL in external_sig_link array

  While here, update url in pkg-descr

  PR:		202655
  Submitted by:	Jeremy Baggs <jbaggs2016@gmail.com>

Changes:
  head/security/base/Makefile
  head/security/base/files/patch-base__conf.php.dist
  head/security/base/pkg-descr
Comment 8 Steve Wills freebsd_committer freebsd_triage 2018-07-10 16:04:21 UTC 
Committed, thanks!