-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Unsupported Confluence API call" when using versions >2.6.0 #1094
Comments
Thanks for the detailed report; it is appreciated. Also, sorry for missing the initial request for help. The logs provided did help with one thing so far. Logging with:
Was not properly logging request JSON data to the standard output stream. I've added a change to address this (#1095). If you are willing, mind trying with a development installation with logging enabled:
Hopefully this will reveal the specific contents of that last |
No worries @jdknight! Thanks for looking into it. I just re-run using the current Last 2 requests
|
Thanks for the logs. Looking at the output, I do not see anything obvious that would indicate that the Browsing online, this appears to be related to CONFCLOUD-72142. I cannot guarantee that this is the error maps to the
Replacing |
Interesting. I have checked both URLs for the property and it might be related to this duplication issue you mention, but it only seems to be visible in v1. https://YOURSITE.atlassian.net/wiki/api/v2/pages/YOURID/properties (single value)
https://YOURSITE.atlassian.net/wiki/rest/api/content/YOURID/property (duplicated values)
I am guessing this behaviour is just an ordering issue, so they are just showing the lowest value id in v1 and highest in v2 or something like that, and not some sort of default value mismatch between v1 and v2? I also tested https://YOURSITE.atlassian.net/wiki/api/v2/pages/YOURID/properties/REDACTED_1
https://YOURSITE.atlassian.net/wiki/api/v2/pages/YOURID/properties/REDACTED_2
Not sure there is an easy way forward to test this? I was thinking maybe deleting one of the ids, but not sure there's a safe(r) value to get rid of? |
Great find, especially the discrepancies between what the v1 API states and the v2 API! Thanks for testing it out. Without trying to speculate on how the situation occurred, I would imagine this is an undesired persistence state in Confluence. What to do next will vary on how you want to approach this. If in a state that's blocking what you want to do, I would imagine you could either just delete the page and have a new one re-create it during a publish event (in hopes the property will be cleared). If history is desired, you should be able to use the API in a way to remove the property entries. I would assume removing one or all of these properties should be fine and Confluence can recover itself -- at worst, the page may switch from a full-width view to a stock view, which you can then re-force the view manually or via a publish event. However, this only corrects the issue on that specific page. Painful if it's a slew of pages that need finessing. And if it happens again, would require a repeat. And if others experience this, they would also have to do a similar approach. What I could do, if willing to wait/test things (since I do not have an instance myself with this scenario -- although I'll try to trigger the bug on my own space later), I can make a modification to this extension that attempts to cleanup the duplicate property if detected. I can place the logic on a test branch and you could try that to see if it cleans up your space's state -- and in theory, this should not be an issue for anyone using this extension. |
That sounds good, I'd be happy to wait. My worry when filing this issue was that it was mentioned in the previous thread that v1 was deprecated and was going to be slowly disappearing, so I wanted to bump to a newer version to ensure the library continued to work. Everything is still working for us in our current version, so I'm ok waiting for a more permanent fix to be included, and of course happy to test it out! By the way, I am guessing you are going to try to work around reading the info from the v1 URL? As I mentioned before, I thought that v1 is deprecated and was going to disappear, isn't this the case? Just thinking on how the fix might be impacted moving forward. |
Hi team and @jdknight, My scenario:
I found a solution using which we can handle the "Unsupported API call" error while building the code and we won't need to delete the complete page in order to get the error https://github.com/sphinx-contrib/confluencebuilder/blob/main/sphinxcontrib/confluencebuilder/publisher.py Currently we are only handling error which has term "unreconciled", we are not handling error "Unsupported API call" Current Code (refer following line):- I hope this solution is legit and works for everyone. Currently I have done the changes in my installed package to handle the scenario for me, however expecting these changes to be made live in the git library as well. Do let me know if any clarification is required, thanks. |
@PradnyaMhatre, thanks for the report. I believe the issue you mentioned is a bit different than the issue being discussed here. I believe I should have addressed the issue you mentioned in #1096. Feel free to provide any new comments related to that issue in that pull request. |
@hasier, I appreciate the willing to wait. I cannot say I have an exact solution yet. I'm hoping to reproduce your current state by manually issuing some API calls. From there, try to work some sort of means to cleanup (e.g. force clearing the single property on v2 and see if when it's repushed, it will clear itself up). Hoping to avoid v1 calls if possible. If a v1 call is needed, it would mainly be invoked in a background to retry the property update. If the property update fails and then if a v1 call (if needed) fails, it would just be another failure scenario and then looking for an alternative way to address this issue (if it still occurs). The plan is to take some time this upcoming weekend to investigate and will hopefully have something ready for next week. |
This issue is essentially the same as #991, except bumping to version 2.6.0+ did not fix the issue for us (I also wrote a comment there, but figured it might be best to open a new issue).
We have experienced everything massdosage went through exactly as mentioned in the original issue, except for the fact that it is not resolved for us 😅 We are currently pinned at
2.4.0
, which is fine for now, but it'd be good to be able to move to more recent versions. Please see below the debugging logs as requested previously in the original issue.We have not tried using
v1
as I saw in the original issue it is discouraged.Request to fetch property
Report
Running in a
ubuntu-latest
Github Action.Last 2 requests
Using
confluence_publish_debug = "headers_and_data"
The text was updated successfully, but these errors were encountered: