Hi! I've been trying to get various methods from the library to work to no avail:

from pydiscourse import DiscourseClient
client = DiscourseClient(host='https://forums.mywebsite.com', api_username='username_of_admin', api_key='key')
print(client.users())

I get the same exception each time:

Traceback (most recent call last):
  File "test_discourse.py", line 3, in <module>
    print(client.users())
  File "C:\Python37\lib\site-packages\pydiscourse\client.py", line 449, in users
    return self._get("/admin/users/list/{0}.json".format(filter), **kwargs)
  File "C:\Python37\lib\site-packages\pydiscourse\client.py", line 1298, in _get
    return self._request(GET, path, params=kwargs, override_request_kwargs=override_request_kwargs)
  File "C:\Python37\lib\site-packages\pydiscourse\client.py", line 1424, in _request
    raise DiscourseClientError(msg, response=response)
pydiscourse.exceptions.DiscourseClientError: Payload Too Large

Running pip freeze | grep discourse gives me pydiscourse==1.1.0. If I access the /admin/users/list/{0}.json endpoint directly in my browser I get the expected result, however, what seems to happen through the library is that an error HTML page is returned instead. Any pointers?

Send request params in the request body instead of the query string. This avoid a pydiscourse.exceptions.DiscourseClientError: Request-URI Too Large error when updating large amounts of data.

I tested this on update_post which is where I was running into this error but didn't test any additional methods.

HI There @bennylope !! this tool is really fantastic. I"m playing around with it and trying to understand the discourse api. This:

https://github.com/bennylope/pydiscourse/blob/master/pydiscourse/client.py#L481

seems to be the way to grab a list of categories and topics.

Questions For You

1. How do i get a list of categories AND subcategories? i can get a list of categories using

client.categories()

But this doesn't return the subcategories from what i can tell.

2. Id like to get a list of topics within each category. When i run client.category_topics(10) (10 is the ID for the category that i'm interested in) I get:

DiscourseError: Invalid Response, expecting "application/json; charset=utf-8" got "text/html; charset=utf-8"

i've played with providing it the slug and the category name and it doesn't seem to work

just curious if you can help me sort this out. i'm happy to update docstrings and docs if i can figure this out if that is of interest / welcome!

Summary of changes

adds api call for: user gets notifications about a topic.

Checklist

• [x] Changes represent a discrete update
• [x] Commit messages are meaningful and descriptive
• [x] Changeset does not include any extraneous changes unrelated to the discrete change

Per the announcement on Discourse meta, global API rate limits have been introduced to the Discourse API. This change adds a new DiscourseRateLimitedError class and a retry mechanism on receipt of a 429.

https://meta.discourse.org/t/global-rate-limits-in-discourse/78612

Closes: #11

I do have one question: Should the 'settings' (retry_{count,backof}) be defined somewhere easier to change?

In the last month or so Discourse implemented API throttling.

I don't have a lot of detail yet but I was told

We recently added rate limiting to api calls which is why you are now running in to some issues. [...] You are allowed 60 requests per minute, so you shouldn’t have to slow things down too much.

Are you open to building the throttling directly in to _request or would you prefer library users handled it individually? https://github.com/bennylope/pydiscourse/blob/master/pydiscourse/client.py#L1186

At the moment an error along these lines is raised (this is what caused me to ask Discourse).

 pydiscourse.exceptions.DiscourseClientError: We have a daily limit on how many times that action can be taken. Please wait 5 seconds before trying again.

thanks, kk

Hi, I'm currently using your packages from pip (thanks!), but I've had to write a bunch of code to work around missing functionality - mainly related to json . Alvaro Molina Alvarez (@amolinaalvarez) has added a number of features including json support that I'm hoping you will be willing to merge. The diff is substantial but easy to follow and includes support for API calls currently not provided in your tree.

https://github.com/bennylope/pydiscourse/compare/master...amolinaalvarez:master

thanks, kk

Hello!

Thank you for this module.

Is it possible to update the pypi version to include the latest change in master ? Notably, I would need the category_latest_topics.

Thanks again!

Are you aware of any issues when supporting redirects from the Discourse server? In your call to requests.request() you explicitly set allow_redirects to False. This prevents topic lookup by path only (omitting the ID).

I think a number of functions could make ID optional using the server redirect to resolve as needed.

Hey there,

v0.7.0 on PyPI is super-old, and there are a bunch of goodies in the master branch here I'd love to have access to. Any idea when there's a plan for an updated PyPI package?

I tried to follow the instructions provided to connect to my https website. It is failing with **SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:852) **

Here is the sample code as per the instructions provided.

from pydiscourse import DiscourseClient client = DiscourseClient( 'https://discuss.itversity.com', api_username='dgadiraju', api_key='api_key_with_single_user_and_global_permissions')

Summary of changes

Two changes here, happy to split if you would prefer.

• First, with my Discourse instance (v2.8.13 on nginx, in case it makes a difference), I encountered rate-limited responses that did not have a Content-Type header. The first commit makes it so that the code takes these in stride.
• The second commit includes information from the Discourse-Rate-Limit-Error-Code, which makes it easier to debug which limit is encountered. Also, logging happens before waiting, to provide feedback on what is happening.

Checklist

• [x] Changes represent a discrete update (-ish)
• [x] Commit messages are meaningful and descriptive
• [x] Changeset does not include any extraneous changes unrelated to the discrete change

Hi, So the client.posts(topic_id) function seems to fetch only the first 20 posts. Is there a way to get all the posts? My code (without the confidential data) is as below:

client = DiscourseClient('http://example.com', api_username='username', api_key='areallylongstringfromdiscourse')

print(len(client.posts(topic_id).get('post_stream').get('posts'))) - This prints out 20 and returns only 20 posts for all topics with > 20 posts. Where the number of posts is < 20, it prints out that correct number.

My apologies if it is already handled, but I didn't found how to do it in the code.

When I want my discourse users, client.users() only gives me the first 100 users whereas I have more than 100.

I've looked at the doc and it seems that the API doesn't tell that there are more users, but we can use the "page" param to get the next ones.

I guess the solution would be to loop on the pages until the call gives an empty list as a result.

I'll try to do a PR if I manage to find a solution.