[Django] #26401: Allow auth machinery to be used without installing auth app

45 Aufrufe
Direkt zur ersten ungelesenen Nachricht

Django

ungelesen,
23.03.2016, 14:09:5323.03.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+--------------------
Reporter: satchamo | Owner: nobody
Type: Bug | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Keywords: auth
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 1 | UI/UX: 0
------------------------------+--------------------
Django projects should be able to use the auth machinery (like
django.contrib.auth.middleware.AuthenticationMiddleware and
django.contrib.auth.context_processors.auth) without having
django.contrib.auth in INSTALLED_APPS

See

https://groups.google.com/forum/#!topic/django-developers/2DzLBbk8w-w

This ticket is for resolving the current issue in 1.9. I also want to
write a test or two to prevent this issue from happening in the future.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

ungelesen,
23.03.2016, 14:10:2823.03.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+--------------------------------------
Reporter: satchamo | Owner: satchamo
Type: Bug | Status: assigned
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:

Keywords: auth | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+--------------------------------------
Changes (by satchamo):

* owner: nobody => satchamo
* status: new => assigned
* needs_docs: => 0
* needs_tests: => 0
* needs_better_patch: => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:1>

Django

ungelesen,
23.03.2016, 18:57:5523.03.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------

Reporter: satchamo | Owner: satchamo
Type: Bug | Status: assigned
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------
Changes (by apollo13):

* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:2>

Django

ungelesen,
05.04.2016, 22:14:4505.04.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: satchamo
Type: Bug | Status: assigned
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------
Changes (by jdufresne):

* has_patch: 0 => 1


Comment:

https://github.com/django/django/pull/6420

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:3>

Django

ungelesen,
09.04.2016, 09:51:1509.04.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: satchamo
Type: Bug | Status: assigned
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------
Changes (by timgraham):

* needs_tests: 0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:4>

Django

ungelesen,
02.06.2016, 13:04:5002.06.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner:

Type: Bug | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------
Changes (by carljm):

* status: assigned => new
* owner: satchamo =>


Comment:

De-assigning so it's clear this is available for a PyCon sprinter to
finish up with tests.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:5>

Django

ungelesen,
02.06.2016, 13:23:3502.06.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: assigned
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------
Changes (by andkon):

* owner: => andkon


* status: new => assigned


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:6>

Django

ungelesen,
14.06.2016, 17:13:5414.06.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: assigned
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------

Comment (by satchamo):

Another patch attempt: https://github.com/django/django/pull/6779

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:7>

Django

ungelesen,
27.08.2016, 08:39:3927.08.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: assigned
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------

Comment (by aaugustin):

FWIW I don't see a lot of value in adding this constraint. Implementing it
requires a fragile and untestable combination of inner imports to avoid
touching the models module.

If the only problem is to avoid creating tables in the database, add a
`MIGRATION_MODULES` entry for the auth app that points to an empty
package. I never tried it but I think it should work.

If this workaround is confirmed to work, I propose to close this ticket as
wontfix.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:8>

Django

ungelesen,
27.08.2016, 12:28:0027.08.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: assigned
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------

Comment (by satchamo):

Interesting idea. It works:
{{{
MIGRATION_MODULES = {
'auth': None
}
}}}

My complaint was driven by the unnecessary clutter in my database when
using a custom user model. So now that there is a reasonable solution, I'm
fine with closing this.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:9>

Django

ungelesen,
27.08.2016, 12:29:0927.08.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: closed
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution: wontfix

Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------
Changes (by satchamo):

* status: assigned => closed
* resolution: => wontfix


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:10>

Django

ungelesen,
28.08.2016, 04:39:0028.08.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: closed
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution: wontfix
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------

Comment (by aaugustin):

Great! Thanks for reporting back.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:11>

Django

ungelesen,
28.08.2016, 19:52:2628.08.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: closed
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution: wontfix
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------

Comment (by jdufresne):

I found the above to be insufficient. Adding `MIGRATION_MODULES` as the
only change to avoid the auth tables fails during migrations with the
following trace:

{{{
Traceback (most recent call last):
File ".../manage.py", line 9, in <module>
execute_from_command_line(sys.argv)
File ".../venv/lib64/python3.5/site-
packages/django/core/management/__init__.py", line 367, in
execute_from_command_line
utility.execute()
File ".../venv/lib64/python3.5/site-
packages/django/core/management/__init__.py", line 359, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File ".../venv/lib64/python3.5/site-
packages/django/core/management/base.py", line 305, in run_from_argv
self.execute(*args, **cmd_options)
File ".../venv/lib64/python3.5/site-
packages/django/core/management/base.py", line 356, in execute
output = self.handle(*args, **options)
File ".../venv/lib64/python3.5/site-
packages/django/core/management/commands/migrate.py", line 222, in handle
self.verbosity, self.interactive, connection.alias,
apps=post_migrate_apps, plan=plan,
File ".../venv/lib64/python3.5/site-
packages/django/core/management/sql.py", line 53, in
emit_post_migrate_signal
**kwargs
File ".../venv/lib64/python3.5/site-
packages/django/dispatch/dispatcher.py", line 191, in send
response = receiver(signal=self, sender=sender, **named)
File ".../venv/lib64/python3.5/site-
packages/django/contrib/auth/management/__init__.py", line 75, in
create_permissions
"content_type", "codename"
File ".../venv/lib64/python3.5/site-packages/django/db/models/query.py",
line 256, in __iter__
self._fetch_all()
File ".../venv/lib64/python3.5/site-packages/django/db/models/query.py",
line 1085, in _fetch_all
self._result_cache = list(self.iterator())
File ".../venv/lib64/python3.5/site-packages/django/db/models/query.py",
line 125, in __iter__
for row in compiler.results_iter():
File ".../venv/lib64/python3.5/site-
packages/django/db/models/sql/compiler.py", line 789, in results_iter
results = self.execute_sql(MULTI)
File ".../venv/lib64/python3.5/site-
packages/django/db/models/sql/compiler.py", line 835, in execute_sql
cursor.execute(sql, params)
File ".../venv/lib64/python3.5/site-
packages/django/db/backends/utils.py", line 79, in execute
return super(CursorDebugWrapper, self).execute(sql, params)
File ".../venv/lib64/python3.5/site-
packages/django/db/backends/utils.py", line 64, in execute
return self.cursor.execute(sql, params)
File ".../venv/lib64/python3.5/site-packages/django/db/utils.py", line
94, in __exit__
six.reraise(dj_exc_type, dj_exc_value, traceback)
File ".../venv/lib64/python3.5/site-packages/django/utils/six.py", line
685, in reraise
raise value.with_traceback(tb)
File ".../venv/lib64/python3.5/site-
packages/django/db/backends/utils.py", line 64, in execute
return self.cursor.execute(sql, params)
django.db.utils.ProgrammingError: relation "auth_permission" does not
exist
LINE 1: ...ntent_type_id", "auth_permission"."codename" FROM "auth_perm...
}}}

This fails becuase the auth's `AppConfig`
[https://github.com/django/django/blob/38cf9ef390eb0cd0703d99893f826153865f6ba6/django/contrib/auth/apps.py#L15-L18
adds a post_migrate callback] that requires the `auth_permission` table to
exist. A project could avoid this by overriding the auth's `AppConfig`. I
think this could be easier for projects if the auth's `AppConfig` was
split into two classes, one base version that can be used without the auth
tables and one that matches the current implementation. This would be a
backwards compatible change that would allow the `MIGRATION_MODULES`
change noted above. For example:

{{{
class BaseAuthConfig(AppConfig):
name = 'django.contrib.auth'
verbose_name = _("Authentication and Authorization")

def ready(self):
checks.register(check_user_model, checks.Tags.models)

class AuthConfig(BaseAuthConfig):
def ready(self):
post_migrate.connect(
create_permissions,
dispatch_uid="django.contrib.auth.management.create_permissions"
)
checks.register(check_models_permissions, checks.Tags.models)
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:12>

Django

ungelesen,
29.08.2016, 05:08:3329.08.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:

Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------
Changes (by claudep):

* status: closed => new
* resolution: wontfix =>


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:13>

Django

ungelesen,
31.08.2016, 00:09:2331.08.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------

Comment (by jdufresne):

[https://github.com/django/django/pull/7188 PR] implementing the
`AppConfig`s described above.

Not sure how to best add a test for this though. Looking for advice.
Details in the PR.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:14>

Django

ungelesen,
01.09.2016, 01:28:2301.09.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 0
------------------------------+------------------------------------
Changes (by jdufresne):

* needs_tests: 1 => 0


Comment:

A test has been added to the PR above.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:15>

Django

ungelesen,
10.09.2016, 08:44:2810.09.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
------------------------------+------------------------------------
Changes (by timgraham):

* needs_better_patch: 0 => 1
* easy: 1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:16>

Django

ungelesen,
10.09.2016, 18:35:2310.09.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
-------------------------------------+-------------------------------------
Reporter: satchamo | Owner: andkon

Type: Bug | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Ready for
| checkin

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by timgraham):

* needs_better_patch: 1 => 0
* stage: Accepted => Ready for checkin


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:17>

Django

ungelesen,
10.09.2016, 19:38:1810.09.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
-------------------------------------+-------------------------------------
Reporter: satchamo | Owner: andkon
Type: Bug | Status: closed
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution: fixed

Keywords: auth | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by GitHub <noreply@…>):

* status: new => closed
* resolution: => fixed


Comment:

In [changeset:"1ec1633cb294d8ce2a65ece6b56c258483596fba" 1ec1633c]:
{{{
#!CommitTicketReference repository=""
revision="1ec1633cb294d8ce2a65ece6b56c258483596fba"
Fixed #26401 -- Added BaseAuthConfig to use auth without migrations.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:18>

Django

ungelesen,
20.09.2016, 21:09:5220.09.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
-------------------------------------+-------------------------------------
Reporter: Matt Johnson | Owner: Andrew
| Konoff
Type: Bug | Status: closed
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution: fixed
Keywords: auth | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Jon Dufresne):

I've hit another snag with the `MIGRATION_MODULES = {'auth': None}`
approach. (Sorry, is it better to reopen this ticket or file a new one?)

The `auth.Permission` model has a fk on `ContentType`. Due to this, when
deleting a `ContentType` (such as with the `remove_stale_contenttypes`
command) queries against the missing `auth_permission` table can occur.

Not sure how best to handle this scenario. IIUC, as long as auth is in
`INSTALLED_APPS`, the models will be loaded and this FK will cause an
issue.

Here is a stack trace when deleting a stale content type

{{{
Traceback (most recent call last):

File "/usr/lib64/python3.5/runpy.py", line 170, in _run_module_as_main
"__main__", mod_spec)
File "/usr/lib64/python3.5/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/jon/devel/django/django/__main__.py", line 9, in <module>
management.execute_from_command_line()
File "/home/jon/devel/django/django/core/management/__init__.py", line
366, in execute_from_command_line
utility.execute()
File "/home/jon/devel/django/django/core/management/__init__.py", line
358, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/home/jon/devel/django/django/core/management/base.py", line 294,
in run_from_argv
self.execute(*args, **cmd_options)
File "/home/jon/devel/django/django/core/management/base.py", line 345,


in execute
output = self.handle(*args, **options)
File

"/home/jon/devel/django/django/contrib/contenttypes/management/commands/remove_stale_contenttypes.py",
line 46, in handle
collector.collect([ct])
File "/home/jon/devel/django/django/db/models/deletion.py", line 218, in
collect
elif sub_objs:
File "/home/jon/devel/django/django/db/models/query.py", line 260, in
__bool__
self._fetch_all()
File "/home/jon/devel/django/django/db/models/query.py", line 1072, in
_fetch_all
self._result_cache = list(self.iterator())
File "/home/jon/devel/django/django/db/models/query.py", line 54, in
__iter__
results = compiler.execute_sql()
File "/home/jon/devel/django/django/db/models/sql/compiler.py", line
847, in execute_sql
cursor.execute(sql, params)
File "/home/jon/devel/django/django/db/backends/utils.py", line 64, in
execute
return self.cursor.execute(sql, params)
File "/home/jon/devel/django/django/db/utils.py", line 94, in __exit__
six.reraise(dj_exc_type, dj_exc_value, traceback)
File "/home/jon/devel/django/django/utils/six.py", line 685, in reraise
raise value.with_traceback(tb)
File "/home/jon/devel/django/django/db/backends/utils.py", line 64, in
execute
return self.cursor.execute(sql, params)
File "/home/jon/devel/django/django/db/backends/sqlite3/base.py", line
334, in execute
return Database.Cursor.execute(self, query, params)
django.db.utils.OperationalError: no such table: auth_permission
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:19>

Django

ungelesen,
20.09.2016, 21:11:1220.09.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
-------------------------------------+-------------------------------------
Reporter: Matt Johnson | Owner: Andrew
| Konoff
Type: Bug | Status: closed
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution: fixed
Keywords: auth | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Jon Dufresne):

* Attachment "ticket-26401-remove-stale-contenttypes.patch" added.

Test case for remove_stale_contenttypes with auth migrations disabled

Django

ungelesen,
20.09.2016, 21:45:4920.09.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+-----------------------------------------

Reporter: Matt Johnson | Owner: Andrew Konoff
Type: New feature | Status: closed
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution: fixed
Keywords: auth 1.11 | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+-----------------------------------------
Changes (by Tim Graham):

* keywords: auth => auth 1.11
* has_patch: 1 => 0
* type: Bug => New feature
* stage: Ready for checkin => Accepted


Comment:

It's appropriate to reopen a ticket if the fix hasn't been released yet.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:20>

Django

ungelesen,
27.09.2016, 15:04:3827.09.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+-----------------------------------------
Reporter: Matt Johnson | Owner: Andrew Konoff
Type: New feature | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:

Keywords: auth 1.11 | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+-----------------------------------------
Changes (by Jon Dufresne):

* status: closed => new

* resolution: fixed =>


Comment:

Reopening until the above issue is handled.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:21>

Django

ungelesen,
19.10.2016, 06:33:1819.10.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+-----------------------------------------
Reporter: Matt Johnson | Owner: Andrew Konoff
Type: New feature | Status: new
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution:
Keywords: auth 1.11 | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+-----------------------------------------
Changes (by Jon Dufresne):

* has_patch: 0 => 1


Comment:

I'm unable to think of a sensible solution to the `ContentType`
`auth.Permission` dependency issue. Therefore, I'm suggesting to revert my
previous PR as using auth without migrations will not fully work.

[https://github.com/django/django/pull/7410 PR to revert]

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:22>

Django

ungelesen,
24.10.2016, 19:50:0324.10.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+-----------------------------------------
Reporter: Matt Johnson | Owner: Andrew Konoff
Type: New feature | Status: new
Component: contrib.auth | Version: 1.9

Severity: Normal | Resolution:
Keywords: auth 1.11 | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+-----------------------------------------

Comment (by Tim Graham):

It seems like you basically want an
`apps.register.Apps.unregister_model()` method or some way of telling
Django not to setup relations for a given model. This might not be
trivial, so if you want to revert for now, that's fine with me.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:23>

Django

ungelesen,
25.10.2016, 20:33:0925.10.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+-----------------------------------------
Reporter: Matt Johnson | Owner: Andrew Konoff
Type: New feature | Status: closed
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution: fixed

Keywords: auth 1.11 | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+-----------------------------------------
Changes (by Jon Dufresne <jon.dufresne@…>):

* status: new => closed
* resolution: => fixed


Comment:

In [changeset:"f3ea0c4bbd5fa9bac73049839ce5de388d53371a" f3ea0c4]:
{{{
#!CommitTicketReference repository=""
revision="f3ea0c4bbd5fa9bac73049839ce5de388d53371a"
Reverted "Fixed #26401 -- Added BaseAuthConfig to use auth without
migrations."

This reverts commit 1ec1633cb294d8ce2a65ece6b56c258483596fba as it
doesn't handle ContentType's auth.Permission dependency. Thus, it
doesn't allow auth without migrations.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:24>

Django

ungelesen,
25.10.2016, 20:34:3025.10.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+-----------------------------------------
Reporter: Matt Johnson | Owner: Andrew Konoff
Type: New feature | Status: new

Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth 1.11 | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+-----------------------------------------
Changes (by Jon Dufresne):

* status: closed => new
* resolution: fixed =>


Comment:

Reopening. The revert commit was merged.

--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:25>

Django

ungelesen,
26.10.2016, 08:08:1226.10.16
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+-----------------------------------------
Reporter: Matt Johnson | Owner: Andrew Konoff
Type: New feature | Status: new
Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+-----------------------------------------
Changes (by Tim Graham):

* keywords: auth 1.11 => auth


* has_patch: 1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:26>

Django

ungelesen,
11.03.2022, 07:00:5811.03.22
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: Matt Johnson | Owner: (none)
Type: New feature | Status: assigned

Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+------------------------------------
Changes (by Mariusz Felisiak):

* owner: Andrew Konoff => (none)


* status: new => assigned


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:27>

Django

ungelesen,
11.03.2022, 07:01:0511.03.22
an django-...@googlegroups.com
#26401: Allow auth machinery to be used without installing auth app
------------------------------+------------------------------------
Reporter: Matt Johnson | Owner: (none)
Type: New feature | Status: new

Component: contrib.auth | Version: 1.9
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+------------------------------------
Changes (by Mariusz Felisiak):

* status: assigned => new


--
Ticket URL: <https://code.djangoproject.com/ticket/26401#comment:28>

Allen antworten
Antwort an Autor
Weiterleiten
0 neue Nachrichten