Apache Allura™

• milestone: limbo --> forge-backlog

• Description has changed:

Diff:

--- old
+++ new
@@ -1,8 +1,8 @@
 Create an Allura Application that stores shortened URLs.  The `short_name` of the URL is given by the user, and the URL can be made either public or private.  We will also need a migration script, which will be created in another ticket, to pull existing URL data from a MySQL database.

-**Example URL** (if installed on `/p/` neighborhood with a `short_name` of `'sshfp'`):
+**Example URL** (if installed on `myproject` at mount point `url` with a URL `short_name` of `'sshfp'`):

-* `http://sf.net/p/url/sshfp`
+* `http://sf.net/p/myproject/url/sshfp`

 **URL Data:**

@@ -43,4 +43,4 @@
 * `created`
 * `last_updated`

-The `Add` and `Browse` pages will need not conflict with user-defined short_names, perhaps by overloading `index` via query paramaters (e.g., `/p/url/?action=browse`).
+The `Add` and `Browse` pages will need not conflict with user-defined short_names, perhaps by overloading `index` via query paramaters (e.g., `/p/myproject/url/?action=browse`).  Or `/p/myproject/url` with no param is used for browsing, and adding is via the admin controller `/p/myproject/admin/url/add`

• labels: --> 42cc

• status: open --> in-progress

created #146: [#4637] Create URL shortener Allura App (4cp)

Closed #146. Branch 42cc_4637.

• status: in-progress --> code-review

• qa: Cory Johns

I've been working on [#4785] and made some fixes to ShortUrl model that related to this ticket, so I updated the code in branch 42cc_4637 with those fixes (commit b6270002882d58bbb34b33850a6c11a1a4083843)

• status: code-review --> open

The ShortUrl model should inherit from Artifact and add an index method, so that the URLs are searchable. Additionally, this ties the model to the project via the AppConfig instance, so the project_id field is unnecessary (using instead the app_config_id and app properties). Also, uninstalling the app should remove the URLs data for that app instance.

Also, I was mistaken about how the hidden field works, as it appears to only hide the tool from the list of installed tools on the Admin page, preventing access to the permissions and other configuration. What we actually need is a browse permission that controls whether the app shows up in the navigation bar by way of the app implementing the is_visible_to method (see Allura/allura/ext/admin/admin_main.py line 69). By default, only Admins should be able to browse the URLs.

Created #168: [#4637] URL shortener Allura App improvements (1cp)

• status: open --> in-progress

Closed #168. All changes in branch 42cc_4637.

• status: in-progress --> code-review

• status: code-review --> closed
• milestone: forge-backlog --> forge-oct-05

I made some corrections to the private flag and search, but otherwise good.

• status: closed --> open
• milestone: forge-oct-05 --> forge-backlog

Discovered some more items that need to be addressed:

• ShortURLAdminController.add() doesn't enforce CREATE/UPDATE permissions, allowing anyone to add or modify URLs if they can construct the POST request without the UI.
• Update needs to be separated from add, and add should not overwrite existing URLs without warning or confirmation
• Need remove functionality

Should probably add Update / Remove links to the table.

Please note that I made some changes as well, so you'll need to rebase from dev.

Created #176: [#4637] URL shortener improvements (2cp)

• status: open --> in-progress

Closed #176. Changes are in branch 42cc_4637a.

• status: in-progress --> code-review

• status: code-review --> closed
• milestone: forge-backlog --> forge-oct-05