63d1a78cd2
Remove capturing parens and debug hexes.
2013-11-20 17:53:25 -06:00
637ce058f5
Write a nonstupid regex (2-pass test)
2013-11-20 17:47:19 -06:00
0ec9881a22
Fix stdout/stderr check to avoid ruby payloads
...
[SeeRM #8498 ]
This knocks out all the non-datastore editing ERROR messages, so we've
got that going for us. Which is nice.
2013-11-20 17:39:35 -06:00
5ef6c5bb44
Land #2668 , avoid tidying nonfiles.
2013-11-20 16:57:57 -06:00
b75f5a8f45
Avoid crashing when msftidy'ing missing files
2013-11-20 16:36:07 -06:00
6c7a98ef47
Be more exact about shebang checking
2013-11-20 15:26:35 -06:00
2c485c509e
Fix caps on module titles (first pass)
2013-11-15 00:03:42 -06:00
2572d8daad
Add #! check to msftidy
2013-11-08 16:11:48 -06:00
bcc9c760c4
Add +x check to msftidy
2013-11-05 11:50:28 -06:00
079c82d11d
Land #2565 - Show full path in msftidy
2013-10-22 16:05:56 -05:00
33c3167362
Show full path instead of just the basename
...
Since @todb-r7 and I hate having to use find. :/
2013-10-22 14:54:54 -05:00
36a7d02001
Update msftidy to check new ZDI reference
2013-10-21 15:31:37 -05:00
07ab53ab39
Merge from master to clear conflict
...
Conflicts:
modules/exploits/windows/brightstor/tape_engine_8A.rb
modules/exploits/windows/fileformat/a-pdf_wav_to_mp3.rb
2013-10-17 13:29:24 -05:00
3fc1a75a6b
Simplify msftidy with Find.find and add fixed()
...
Also, enforce binary encoding like the other Metasploit tools.
This opens the door to fixing files that have things that could be fixed
programmatically.
[SeeRM #8497 ]
2013-10-16 10:40:42 -05:00
81d145ad81
At least offer a solution with msftidy
...
I would go ahead and fix it for the user, but due to #8497 , I can't
yet.
2013-10-15 13:53:38 -05:00
e9e6fb7e26
Add msftidy check.
2013-10-15 13:35:52 -05:00
36d058b28c
Warn for tabbed indentation
2013-10-01 12:22:46 -05:00
2e8d19edcf
Retab all the things (except external/)
2013-09-30 13:47:53 -05:00
8fe9132159
Land #2358 , deprecate funny names
2013-09-18 14:55:33 -05:00
32d2f7ffce
Hard tabs for now
2013-09-12 16:15:50 -05:00
52843c6a67
Revert whitespace change to msf_tidy.rb
...
Causing merge conflicts, I'll re-tab it after this PR lands.
This reverts commit 1178da46c6
2013-09-12 16:14:42 -05:00
1178da46c6
Normalize indentation or @wchen-r7 will be cross
2013-09-12 16:10:43 -05:00
cf27b0b457
Add msftidy check for snake_case.rb filenames
2013-09-12 16:06:17 -05:00
40e7f45db4
another regex fix
2013-08-30 16:10:16 +02:00
921ec615c7
Bugfix
2013-08-29 21:35:15 +02:00
1839af4b89
check for modified datastore
2013-08-29 07:31:17 +02:00
ef224b175d
Allow for tabs or spaces as indentation
...
This signals a move to allowing for normal Ruby indentation (2 space
soft tabs). This change will check files for indentation of spaces or of
tabs, since we don't want to fail out all modules quite yet.
For more, see
https://github.com/rapid7/metasploit-framework/wiki/Indentation-Standards
where all details of the conversion plan will be documented in order to
minimize the amount of whitespace conflict we are sure to encounter over
this conversion.
2013-08-07 11:45:46 -05:00
04c2a7367d
Uncapitalized function names are rarely improper.
2013-07-01 15:37:22 -05:00
4e42ffd51e
msftidy cleanup
2013-04-12 21:39:11 +02:00
f6da02d907
Check for VERBOSE Option
2013-04-12 21:34:15 +02:00
fd20eba35e
Expanding the title and desc for external_ip
...
Also allowing the capitalization on "via" to be small.
2013-03-20 14:42:12 -05:00
45db43d2b3
Merge branch 'msftidy/no-twitter-handles' of github.com:todb-r7/metasploit-framework into todb-r7-msftidy/no-twitter-handles
2013-02-04 14:21:40 -06:00
e8def29b4f
Dropping all twitter handles
...
Also adds "pbot" as an accepted lowercase word. This will come up pretty
routinley for functions and stuff.
2013-02-01 16:33:52 -06:00
7b6d1f4fdd
Actually test alternate rubies.
2013-02-01 13:36:15 -06:00
bf013ba65f
Add more words to ignore
2013-01-10 01:54:19 -06:00
acdce4c876
Correct unicode probs
2013-01-04 14:09:37 -06:00
3dc051c296
Remove return
2012-12-14 11:07:41 -06:00
4011c8dba7
Ignore refs commented out
2012-12-14 10:17:10 -06:00
3297db7bf7
Add function check_ref_identifiers
2012-12-13 17:13:53 -06:00
ab26995910
Add more words to ignore
2012-12-13 13:21:34 -06:00
9a2cb7eddf
Improve title casing check
...
Sometimes the first letter of a word shouldn't be capitalized.
If you do, it may actually be technically incorrect. For example:
a function name, a filename, or even a software name like freeFTPd.
We should ignore scenarios like those.
2012-12-13 13:07:23 -06:00
d90d41d706
Allow version or build numbers
2012-12-13 12:47:36 -06:00
438c18472e
damn it, typo
2012-12-13 12:44:15 -06:00
d8b78868c9
Change case checking code
2012-12-13 12:43:12 -06:00
cdb47bfcfc
Forgot to uncomment back
2012-12-13 12:31:03 -06:00
6f747f16a0
Add another word to ignore
2012-12-13 12:26:32 -06:00
0c84cd0d01
Ignore function names
2012-12-13 12:22:52 -06:00
a049327f21
Ignore certain words
2012-12-13 12:21:22 -06:00
051fe4ddd4
Make exceptions for check_title_format
...
It's not always possible or technically correct to uppercase every
word. For example, sometimes a module may contain a filename or
function name. We should allow these conditions to pass.
2012-12-13 12:11:27 -06:00
014269cd25
Small changes
2012-12-12 15:47:04 -06:00
Tod Beardsley
William Vu
sinn3r
Tab Assassin
James Lee
Christian Mehlmauer
h0ng10
Tod Beardsley
sinn3r