Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Headers update #27

Merged
merged 13 commits into from
Apr 24, 2020
Merged

Conversation

5pitf1re
Copy link
Contributor

This addresses Discourse latest api changes: https://meta.discourse.org/t/discourse-api-documentation/22706/1

On April 6th, 2020 we dropped support for all non-HTTP header based authentication (excluding some rss, mail-receiver, and ics routes). This means that API requests that have an api_key and api_username in the query params or in the HTTP body of the request will soon stop working. Please see the example cURL request below for how to update your API requests to use the HTTP headers for authentication.

Octavian A. Damiean and others added 12 commits April 14, 2020 13:59
Changed every HTTP request to send api_key and api_username in the header of each request due to changes to Discourse.
Missed a file because I'm bad.
Updated braces to psr2_braces
I had to fix the headers again after being bad and not paying attention.
Removed length_ordered_imports
Yet another change to make sytleci happy
for the love of god ...
Last try or I'll flip the table
This one time it was correct ...
if this fails, styleci is fucked and should be fixed
fixed the missing comma there
@MichaelErmer MichaelErmer mentioned this pull request Apr 16, 2020
@herpaderpaldent herpaderpaldent self-requested a review April 16, 2020 20:51
Fixed a case error
@herpaderpaldent herpaderpaldent mentioned this pull request Apr 17, 2020
herpaderpaldent added a commit that referenced this pull request Apr 24, 2020
This PR contains changes from @5pitf1re and @MichaelErmer from #28 and #27 making style.ci happy.
exclamation ATTENTION

This package is one of my first work and no testing setup, therefore manual testing is required which i can currently not execute by myself i am fully dependent on the contributors of the changes for the functionality.
Other then that, i can not guarantee functionality in seat 4.x if it is ever released.
@herpaderpaldent herpaderpaldent merged commit fcb5487 into herpaderpaldent:master Apr 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants