[ 
https://issues.apache.org/jira/browse/SHINDIG-27?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chak Nanga updated SHINDIG-27:
------------------------------

    Attachment: blacklist_patch.txt

Notes for the attached patch:

Introduced a basic GadgetBlacklistChangeListener interface and modified the 
BasicGadgetBlacklist impl to listen for black list change notifications. Uses 
simple file modification timestamps to refresh the blacklist.

The blacklist file (as it's currently specified in the web.xml file) should be 
in a dir named "conf". The "conf" dir is expected to be at the same level as 
WEB-INF. Of course, the location can be changed via the conf. file.


> Hookup Gadget blacklist implementation to the Gadget Server
> -----------------------------------------------------------
>
>                 Key: SHINDIG-27
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-27
>             Project: Shindig
>          Issue Type: Improvement
>          Components: Gadgets Server - Java
>            Reporter: Chak Nanga
>            Assignee: John Hjelmstad
>            Priority: Minor
>         Attachments: blacklist_patch.txt
>
>
> Currently there is a BasicGadgetBlacklist class, however, it's not being 
> utilized (i.e. being init'd and read in) by the Gadget Server. While we're at 
> it, we should also implement a mechanism to be able to dynamically update the 
> blacklist file without having to restart the server.
> Proposed solution:
> 1. Read in the black list file name from web.xml (as a context-param)
> 2. Implement a file change listener mechanism that can be used to monitor 
> changes to the blacklist file and reload it on changes.
> I can work on the patch if you agree on the proposed solution

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to