Issue #4091
closed
The master branch is failing
Description
Example:
https://travis-ci.org/pulp/pulp/builds/443310599
I believe it's probably due to the new version of DRF (3.9.0) which just came out today.
Updated by dalley about 6 years ago
- Triaged changed from No to Yes
- Sprint set to Sprint 44
- Tags Pulp 3 added
DRF 3.9 release notes
https://www.django-rest-framework.org/community/3.9-announcement/
Updated by daviddavis about 6 years ago
Looks like with DRF 3.8, you get a json response when you hit the base url (/pulp/api/v3/). Now with 3.9, we get:
info:
description: ''
title: Pulp API
version: ''
openapi: 3.0.0
paths: {}
servers:
- url: http://localhost:8000/pulp/api/v3/
pulp-smash is hitting this base url to test authentication and they are expecting back json.
From the release notes that dalley posted:
The get_schema_view(...) method now defaults to OpenAPI YAML, with CoreJSON as a secondary option if it is selected via HTTP content negotiation.
Updated by daviddavis about 6 years ago
I don't think using the base_url is a good test for authentication. It doesn't require a user necessarily and it's not something we've intentionally created in our application. I think something like /pulp/api/v3/users/ might be better.
Updated by daviddavis about 6 years ago
- Status changed from NEW to POST
- Assignee set to daviddavis
Added by daviddavis about 6 years ago
Added by daviddavis about 6 years ago
Revision 9c29ece0 | View on GitHub
Use /pulp/api/v3/users to test auth
I don't think using the /pulp/api/v3/ is a good test for authentication. It doesn't require a user necessarily and it's not something we've intentionally created in our application. I think something like /pulp/api/v3/users/ might be better.
Updated by daviddavis about 6 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|9c29ece073fdbe54d01e2d5b41c524f93e913dc8.
Updated by bmbouter almost 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Use /pulp/api/v3/users to test auth
I don't think using the /pulp/api/v3/ is a good test for authentication. It doesn't require a user necessarily and it's not something we've intentionally created in our application. I think something like /pulp/api/v3/users/ might be better.
fixes #4091 https://pulp.plan.io/issues/4091