You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@allura.apache.org by Dave Brondsema <br...@users.sf.net> on 2014/01/07 00:10:06 UTC

[allura:tickets] #7029 AuthProvider should be able to add routes to /auth/



---

** [tickets:#7029] AuthProvider should be able to add routes to /auth/**

**Status:** in-progress
**Labels:** 42cc 
**Created:** Mon Jan 06, 2014 11:10 PM UTC by Dave Brondsema
**Last Updated:** Mon Jan 06, 2014 11:10 PM UTC
**Owner:** nobody

Some auth providers may want to add additional user configuration pages under `/auth/`.  The `AuthenticationProvider` class already allows a lot of configuration, but can't add new url handling.  We should extend it to have a new method which may return references to methods and/or controllers to add.  Then `AuthController.__init__` can just loop through those and use `setattr()` to add them to itself.  Default implementation should `return []` (so existing providers work fine) and have a docstring.


---

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #7029 AuthProvider should be able to add routes to /auth/

Posted by Igor Bondarenko <je...@users.sf.net>.
Closed #521. `je/42cc_7029`

For QA: you can add something like the following to `LocalUserPreferencesProvider`:

    :::python
    @expose()
    def add(self):
        return 'Hello!'

    def additional_urls(self):
        return [('add', self.add), ]


Then go to `/auth/add` and you should see 'Hello!' there.


---

** [tickets:#7029] AuthProvider should be able to add routes to /auth/**

**Status:** code-review
**Labels:** 42cc 
**Created:** Mon Jan 06, 2014 11:10 PM UTC by Dave Brondsema
**Last Updated:** Wed Jan 08, 2014 06:57 PM UTC
**Owner:** nobody

Some auth providers may want to add additional user configuration pages under `/auth/`.  The `AuthenticationProvider` class already allows a lot of configuration, but can't add new url handling.  We should extend it to have a new method which may return references to methods and/or controllers to add.  Then `AuthController.__init__` can just loop through those and use `setattr()` to add them to itself.  Default implementation should `return []` (so existing providers work fine) and have a docstring.


---

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #7029 AuthProvider should be able to add routes to /auth/

Posted by Igor Bondarenko <je...@users.sf.net>.
- **status**: in-progress --> code-review



---

** [tickets:#7029] AuthProvider should be able to add routes to /auth/**

**Status:** code-review
**Labels:** 42cc 
**Created:** Mon Jan 06, 2014 11:10 PM UTC by Dave Brondsema
**Last Updated:** Wed Jan 08, 2014 06:57 PM UTC
**Owner:** nobody

Some auth providers may want to add additional user configuration pages under `/auth/`.  The `AuthenticationProvider` class already allows a lot of configuration, but can't add new url handling.  We should extend it to have a new method which may return references to methods and/or controllers to add.  Then `AuthController.__init__` can just loop through those and use `setattr()` to add them to itself.  Default implementation should `return []` (so existing providers work fine) and have a docstring.


---

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #7029 AuthProvider should be able to add routes to /auth/

Posted by Cory Johns <ma...@users.sf.net>.
- **status**: code-review --> closed
- **Milestone**: forge-backlog --> forge-mar-7



---

** [tickets:#7029] AuthProvider should be able to add routes to /auth/**

**Status:** closed
**Labels:** 42cc 
**Created:** Mon Jan 06, 2014 11:10 PM UTC by Dave Brondsema
**Last Updated:** Thu Feb 27, 2014 08:07 PM UTC
**Owner:** nobody

Some auth providers may want to add additional user configuration pages under `/auth/`.  The `AuthenticationProvider` class already allows a lot of configuration, but can't add new url handling.  We should extend it to have a new method which may return references to methods and/or controllers to add.  Then `AuthController.__init__` can just loop through those and use `setattr()` to add them to itself.  Default implementation should `return []` (so existing providers work fine) and have a docstring.


---

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #7029 AuthProvider should be able to add routes to /auth/

Posted by Dave Brondsema <br...@users.sf.net>.
`UserPreferencesProvider` might be slightly better than `AuthenticatonProvider` since this is more about additional preferences.    


---

** [tickets:#7029] AuthProvider should be able to add routes to /auth/**

**Status:** in-progress
**Labels:** 42cc 
**Created:** Mon Jan 06, 2014 11:10 PM UTC by Dave Brondsema
**Last Updated:** Mon Jan 06, 2014 11:10 PM UTC
**Owner:** nobody

Some auth providers may want to add additional user configuration pages under `/auth/`.  The `AuthenticationProvider` class already allows a lot of configuration, but can't add new url handling.  We should extend it to have a new method which may return references to methods and/or controllers to add.  Then `AuthController.__init__` can just loop through those and use `setattr()` to add them to itself.  Default implementation should `return []` (so existing providers work fine) and have a docstring.


---

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #7029 AuthProvider should be able to add routes to /auth/

Posted by Cory Johns <ma...@users.sf.net>.
- **QA**: Cory Johns



---

** [tickets:#7029] AuthProvider should be able to add routes to /auth/**

**Status:** code-review
**Labels:** 42cc 
**Created:** Mon Jan 06, 2014 11:10 PM UTC by Dave Brondsema
**Last Updated:** Tue Feb 25, 2014 08:55 AM UTC
**Owner:** nobody

Some auth providers may want to add additional user configuration pages under `/auth/`.  The `AuthenticationProvider` class already allows a lot of configuration, but can't add new url handling.  We should extend it to have a new method which may return references to methods and/or controllers to add.  Then `AuthController.__init__` can just loop through those and use `setattr()` to add them to itself.  Default implementation should `return []` (so existing providers work fine) and have a docstring.


---

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.