getFeed implementation does not support public feeds
- Dominant language
- JavaScript
- Stars
- 7
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Description
I am trying to implement support for ChRIS_ui to be able to display public feeds. Currently, CUBE allows a client to get a feed without authentication, but searching (or listing) requires authentication.
The implementation of `getFeed` is problematic because the request sent is `api/v1/search/?id=N`. A better solution which would also work for public feeds would be for the request to get `api/v1/N/`.
https://github.com/FNNDSC/fnndsc/blob/dcacf64534df90dcb1ef7d1c2f69000bd437287a/js/chrisAPI/src/client.js#L184-L186
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in js/chrisAPI/src/client.js at the getFeed implementation referenced around lines 184–186. Trace how the request URL is constructed and verify the behavior for a public feed. Done means getFeed uses the feed-specific api/v1/N/ endpoint so public feeds can be retrieved without authentication.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- api
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100