diff options
author | Nicolas LÅ“uillet <nicolas@loeuillet.org> | 2017-05-01 09:21:59 +0200 |
---|---|---|
committer | Nicolas LÅ“uillet <nicolas@loeuillet.org> | 2017-05-01 09:26:12 +0200 |
commit | efd351c98fa0caa4c8df9c7ff6965c537524f12a (patch) | |
tree | 78a268e63d8983b88c95a2518df62b9b06889827 /src | |
parent | 7fa844a34983f9929348e70ddd408cf6ba5811b6 (diff) | |
download | wallabag-efd351c98fa0caa4c8df9c7ff6965c537524f12a.tar.gz wallabag-efd351c98fa0caa4c8df9c7ff6965c537524f12a.tar.zst wallabag-efd351c98fa0caa4c8df9c7ff6965c537524f12a.zip |
Added limit
Diffstat (limited to 'src')
3 files changed, 13 insertions, 0 deletions
diff --git a/src/Wallabag/ApiBundle/Controller/EntryRestController.php b/src/Wallabag/ApiBundle/Controller/EntryRestController.php index ae6f0e3f..7c3e778e 100644 --- a/src/Wallabag/ApiBundle/Controller/EntryRestController.php +++ b/src/Wallabag/ApiBundle/Controller/EntryRestController.php | |||
@@ -5,6 +5,7 @@ namespace Wallabag\ApiBundle\Controller; | |||
5 | use Hateoas\Configuration\Route; | 5 | use Hateoas\Configuration\Route; |
6 | use Hateoas\Representation\Factory\PagerfantaFactory; | 6 | use Hateoas\Representation\Factory\PagerfantaFactory; |
7 | use Nelmio\ApiDocBundle\Annotation\ApiDoc; | 7 | use Nelmio\ApiDocBundle\Annotation\ApiDoc; |
8 | use Symfony\Component\Config\Definition\Exception\Exception; | ||
8 | use Symfony\Component\HttpFoundation\Request; | 9 | use Symfony\Component\HttpFoundation\Request; |
9 | use Symfony\Component\HttpFoundation\JsonResponse; | 10 | use Symfony\Component\HttpFoundation\JsonResponse; |
10 | use Symfony\Component\Routing\Generator\UrlGeneratorInterface; | 11 | use Symfony\Component\Routing\Generator\UrlGeneratorInterface; |
@@ -229,6 +230,8 @@ class EntryRestController extends WallabagRestController | |||
229 | * ) | 230 | * ) |
230 | * | 231 | * |
231 | * @return JsonResponse | 232 | * @return JsonResponse |
233 | * | ||
234 | * @throws Symfony\Component\Config\Definition\Exception\Exception When limit is reached | ||
232 | */ | 235 | */ |
233 | public function postEntriesListAction(Request $request) | 236 | public function postEntriesListAction(Request $request) |
234 | { | 237 | { |
@@ -237,6 +240,12 @@ class EntryRestController extends WallabagRestController | |||
237 | $urls = json_decode($request->query->get('urls', [])); | 240 | $urls = json_decode($request->query->get('urls', [])); |
238 | $results = []; | 241 | $results = []; |
239 | 242 | ||
243 | $limit = $this->container->getParameter('wallabag_core.api_limit_mass_actions'); | ||
244 | |||
245 | if (count($urls) > $limit) { | ||
246 | throw new Exception('API limit reached'); | ||
247 | } | ||
248 | |||
240 | // handle multiple urls | 249 | // handle multiple urls |
241 | if (!empty($urls)) { | 250 | if (!empty($urls)) { |
242 | foreach ($urls as $key => $url) { | 251 | foreach ($urls as $key => $url) { |
diff --git a/src/Wallabag/CoreBundle/DependencyInjection/Configuration.php b/src/Wallabag/CoreBundle/DependencyInjection/Configuration.php index 006a18c3..75b37729 100644 --- a/src/Wallabag/CoreBundle/DependencyInjection/Configuration.php +++ b/src/Wallabag/CoreBundle/DependencyInjection/Configuration.php | |||
@@ -47,6 +47,9 @@ class Configuration implements ConfigurationInterface | |||
47 | ->scalarNode('list_mode') | 47 | ->scalarNode('list_mode') |
48 | ->defaultValue(1) | 48 | ->defaultValue(1) |
49 | ->end() | 49 | ->end() |
50 | ->scalarNode('api_limit_mass_actions') | ||
51 | ->defaultValue(10) | ||
52 | ->end() | ||
50 | ->end() | 53 | ->end() |
51 | ; | 54 | ; |
52 | 55 | ||
diff --git a/src/Wallabag/CoreBundle/DependencyInjection/WallabagCoreExtension.php b/src/Wallabag/CoreBundle/DependencyInjection/WallabagCoreExtension.php index aa9ee339..c075c19f 100644 --- a/src/Wallabag/CoreBundle/DependencyInjection/WallabagCoreExtension.php +++ b/src/Wallabag/CoreBundle/DependencyInjection/WallabagCoreExtension.php | |||
@@ -26,6 +26,7 @@ class WallabagCoreExtension extends Extension | |||
26 | $container->setParameter('wallabag_core.action_mark_as_read', $config['action_mark_as_read']); | 26 | $container->setParameter('wallabag_core.action_mark_as_read', $config['action_mark_as_read']); |
27 | $container->setParameter('wallabag_core.list_mode', $config['list_mode']); | 27 | $container->setParameter('wallabag_core.list_mode', $config['list_mode']); |
28 | $container->setParameter('wallabag_core.fetching_error_message', $config['fetching_error_message']); | 28 | $container->setParameter('wallabag_core.fetching_error_message', $config['fetching_error_message']); |
29 | $container->setParameter('wallabag_core.api_limit_mass_actions', $config['api_limit_mass_actions']); | ||
29 | 30 | ||
30 | $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config')); | 31 | $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config')); |
31 | $loader->load('services.yml'); | 32 | $loader->load('services.yml'); |