ff2f8a63bb
Fix bugs in purge classes, cleanup - OfflinePlayer objects were never mapped to names - Essentials purge was run even if setting was disabled - Console user saw messages twice - Misc: trivial field reorderings, change DefaultPermission to be null safe - Permissions: do not expose PermissionHandler just to get its type
ljacqu
2016-06-19 17:56:50 +02:00
a0abe3d19d
Minor householding - Remove getter for NewSetting - Reduce visibility of classes used by DelayedRunner
ljacqu
2016-06-19 16:13:00 +02:00
12533a90f6
Join/PreLogin event listener changes
Gabriele C
2016-06-19 10:40:20 +02:00
fb5e7d40c6
Change delayed injection to only change behavior of its own elements - Introduce new BeforeInjecting annotation to not modify the (expected) order of annotations that do not belong to us - Avoids using hacky way of first initializing field to an Answer that will delegate on demand to the proper class... - Remove PostConstruct support for Mockito's InjectMocks: we should not change the established behavior of external elements
ljacqu
2016-06-18 15:31:11 +02:00
a1c62e7c04
Create delayed injection test runner - Test runner supporting new "DelayedInjection" annotation: such fields are only initialized with instantiation right before the first time they're used in tests, allowing to set up mock behavior beforehand
ljacqu
2016-06-18 13:19:07 +02:00
cd1acfde1b#674 Create tests for purge commands and purge service
ljacqu
2016-06-18 11:13:17 +02:00
51663703ea
Merge branch 'master' into 674-purge-process-refactor
Gnat008
2016-06-17 20:51:11 -04:00
16e9990723
implement checking permissions of an offline player
Gnat008
2016-06-17 20:49:03 -04:00
c79857cc14
Add Gnat008 as project author
Gabriele C
2016-06-17 02:18:12 +02:00
ac73d43573
Add Gnat008 to team.txt
Gabriele C
2016-06-17 02:17:02 +02:00
3629c51fc1#674 PurgeService: Always register if purging, reduce code duplication - Rename autoPurging to isPurging: we should always register if a purge task is in progress (regardless if autopurge or not) and deny any new requests - Reduce the same logic being coded multiple times by calling through the methods - DataSource: remove purgeBanned in favor of purgeRecords, both do exactly the same thing
ljacqu
2016-06-16 22:52:11 +02:00
2ac89f5938
Apply minor changes to PurgeService - Remove various imports / unused fields - Make CacheDataSource call source for purging on DB - Minor: SQLite - place creation of PreparedStatement outside of loop - Make specific purge actions called from task package-private (clearer from the outside which methods can be called from the outside)
ljacqu
2016-06-16 21:04:12 +02:00
9753a3af9e
Merge pull request #140 from Xephi/master
Gabriele C
2016-06-16 19:41:37 +02:00
30cdae1c44
Fix links in README
Gabriele C
2016-06-16 19:40:21 +02:00
fb8baeafd2
remove database auto purge methods
Gnat008
2016-06-16 12:52:42 -04:00
fde634e054
Merge branch 'master' into 674-purge-process-refactor
Gnat008
2016-06-16 12:36:31 -04:00
ac484345a2
Change password task to async process - Perform async change password task just like other async processes: via Management - Remove legacy setting - Remove now unused service getter (#736)
ljacqu
2016-06-15 20:56:34 +02:00
15886fb517#723 Let LimboPlayer task manager decide which message key to use - Pass boolean (is registered) value and determine internally which message key (email registration vs. regular) to use in the message task
ljacqu
2016-06-15 20:37:00 +02:00
68c3aabce4
Merge branch 'master' into 674-purge-process-refactor
Gnat008
2016-06-15 13:07:21 -04:00
5870a4a433
Add ability to check permissions by player name
EbonJaguar
2016-06-14 22:47:35 -04:00
5cbb83e153
Code householding, add tests to TempbanManager - Delegate event firing to BukkitService - Write tests for IP banning function - Update comments on tempban properties in config.yml
ljacqu
2016-06-14 21:52:43 +02:00
3411450ff1#760 Fix single session feature - Move the check from PlayerLoginEvent to AsyncPlayerPreLoginEvent. Single session can only be implemented with PreLoginEvent; it is already to late to check this in the PlayerLoginEvent. Ergo, we cannot offer this for CraftBukkit. - Remove interactions with LimboCache - no interactions with LimboCache expected until after OnJoinVerification checks. (Thanks sgdc3!)
ljacqu
2016-06-14 21:03:32 +02:00
367f785610
count login failures by ip address and not by name
EbonJaguar
2016-06-13 15:58:03 -04:00
89bbfc48ee#723 Create provider for TimeoutTask and MessageTask, remove LimboCache#getInstance - Create class to handle the creation of "LimboPlayer tasks" (adds encapsulation, reduces duplication) - Move group setting into its own class because (mutual dependency between LimboCache and PermissionsManager otherwise)
ljacqu
2016-06-12 21:59:26 +02:00
69100daba5
Get jar files via URI objects in tests to prevent issue with spaces
ljacqu
2016-06-12 20:54:39 +02:00
c3d391aeaf
lets only inject in one place (moved injects to constructor)
Gnat008
2016-06-12 14:53:18 -04:00
ac4add9f54
add ability to tempban users after x wrong logins - ref #520#192
Gnat008
2016-06-12 13:40:34 -04:00
d6e1fd5ceb
Use injection in and for LimboCache, migrate some legacy settings, remove setGroup from Utils - New injector method allows to retrieve services if they've already been instantiated -> useful for onDisable() which might be run after aborted initialization - Deprecate various methods that need to be removed
ljacqu
2016-06-12 16:14:06 +02:00
347d7bcf46
Update messages_de.yml
Gabriele C
2016-06-12 14:29:16 +02:00
5ef62784b5#729 Use world from LimboPlayer for "spawn after login" feat., simplify teleport event constructors
ljacqu
2016-06-12 14:10:38 +02:00
3cdec91255
Do not teleport unregistered player if registration is optional
ljacqu
2016-06-12 12:48:32 +02:00
68d5145cd7#729 Refactor spawn handling into separate service (work in progress)
ljacqu
2016-06-12 12:46:03 +02:00
f0e42b61c5#747 - display captcha immediately after the failed login attempt
Gnat008
2016-06-11 21:23:53 -04:00
1de086c090
Fix 755
Gabriele C
2016-06-11 18:05:11 +02:00
01f297919d
1.10 API + cleanup
Gabriele C
2016-06-11 17:46:24 +02:00
2e269b6f5e
Add missing unit tests for commands
ljacqu
2016-06-05 13:21:05 +02:00
a5a796e900
Update coveralls-maven-plugin
Gabriele C
2016-06-05 01:47:17 +02:00
ee23f7242b
Merge pull request #138 from AuthMe-Team/565-skip-long-hash-tests
Gabriele C
2016-06-05 00:45:36 +02:00
3753a0ef96#565 Allow to skip extended encryption methods - Set system property via surefire plugin and create profile that modifies the property - Check for the new property in AbstractEncryptionMethodTest and shorten/skip the tests when necessary
ljacqu
2016-06-05 00:29:31 +02:00
4fe26f08d4#612 Check if plugin is permission system within method itself - Iterate over all values within the method - Bug fix: change method to use pluginName field, and not name
ljacqu
2016-06-04 22:40:30 +02:00
2a4bb483a3#727 finalization - minor javadoc changes
ljacqu
2016-06-04 21:57:31 +02:00
91111ca476#727 Remove CommandService being passed as parameter to other methods - No longer need to pass as param to other methods since CommandService is now an injected field
ljacqu
2016-06-04 21:16:58 +02:00
26ac466035#727 Instantiate ExecutableCommand objects in CommandHandler - Change CommandDescription to contain a reference to ExecutableCommand class only - Instantiate the actual ExecutableCommand objects in CommandHandler
ljacqu
2016-06-04 21:13:38 +02:00
c6778b566d#727 Remove CommandService from ExecutableCommand interface (work in progress) - Inject CommandService like other classes instead of passing it as method parameter - Not solved: cyclic dependency CommandInitializer > ExecutableCommand > CommandService > CommandInitializer...
ljacqu
2016-06-04 11:02:15 +02:00
40ce01f65e#601 Increase captcha count on login start - Increase login count at start of the login process (as done previously) and not only when login has failed AND player is still online - Add missing javadoc to CaptchaManager
ljacqu
2016-06-04 10:44:34 +02:00
12703d1613#601 Integrate plugin manager - Encapsulate captcha functionality into a class instead of two public fields on the AuthMe main class(!) - Let CaptchaManager worry about whether it is enabled or not -> no need to check on the outside - Implement full reloading support to enable/disable captchas + parameters - Add unit tests
ljacqu
2016-06-03 22:47:17 +02:00
e8717ba076
Update verification notes in messages files + add entry for #743 invalid chars in password
ljacqu
2016-06-03 12:52:47 +02:00
55f7e8097a#743 Add proper error message for "invalid chars in password" - Change password validation to return a ValidationResult object for passing message arguments - Remove wrapping methods in ProcessService and CommandService and use ValidationService directly
ljacqu
2016-06-03 12:51:49 +02:00
6549ebbf5e
Should fix#731
Gabriele C
2016-06-03 01:18:54 +02:00
6c0dec887a
try to fix javadoc
Gabriele C
2016-06-03 00:27:06 +02:00
eafb3b6653
cleanup
Gabriele C
2016-06-03 00:07:25 +02:00
6d2597a980#739 Fix command unit tests; adjust default handling (cherry picked from commit 61c119b)
ljacqu
2016-06-02 23:08:13 +02:00
73272b5931
Remove all but one hasPermission() method in the PermissionsManager #739 (cherry picked from commit 65f3347)
Gnat008
2016-06-02 22:35:07 +02:00
c3d07cb9a4#739 Cleanup on PermissionsManager (cherry picked from commit d9ad12b)
EbonJaguar
2016-06-02 02:49:10 +02:00
9b5009eb8c#742 Create test that plugin.yml corresponds to command definitions - Create test - Fix definitions to correspond
ljacqu
2016-06-02 15:49:21 +02:00
e75cff5fb8
Remove static injections in ListenerService - Get other classes via Inject annotation - Remove no longer needed Utils methods (relates to #736) - Create tests for ListenerService and AuthMeBlockListener - Performance improvement: keep unrestricted names as Set instead of List for faster contains() method
ljacqu
2016-06-02 12:46:54 +02:00
2581b95afb
Fix NPE on server stop #740
DNx
2016-06-02 11:07:37 +07:00
e06c5e7309
Fix NPE from ConsoleLogger
DNx
2016-06-02 05:23:16 +07:00
408e8dd0dd#739 Create unit tests for PermissionsManager#hasPermission
ljacqu
2016-06-02 00:03:02 +02:00
38db2ef0bd
Minor code householding - Add removed property to migration service - Log exception with appropriate logger method
ljacqu
2016-06-01 23:24:48 +02:00
95343e366b#314 Evaluate default permission on enum, remove default from handler interface - Evaluate permission for DefaultPermission on the enum itself - Remove boolean default from PermissionHandler for hasPermission() - Remove some unused / intermediary hasPermission() flavors in PermissionsManager
ljacqu
2016-06-01 21:56:57 +02:00
fdb9227ec1
Properly unhook the current system when unhook() is called
EbonJaguar
2016-06-01 14:09:30 -04:00
e99d9414b8
Add and implement handlers for each individual permissions plugin, instead of doing everything in the PermissionsManager - see #314
EbonJaguar
2016-06-01 14:00:27 -04:00
1cbd11a753
Fix the issue reference for removing the exception stacktrace logging
games647
2016-06-01 18:30:46 +02:00
ljacqu