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

fix: missing headers on search endpoint with token #121

Merged
merged 1 commit into from
Aug 25, 2019
Merged

Conversation

juanpicado
Copy link
Member

Type: bug

Description:

Headers should be part of the options if we override options.

https://github.com/verdaccio/ui/issues/118

Resolves #118

@juanpicado juanpicado changed the title fix: missing headers on search endopoint with token fix: missing headers on search endpoint with token Aug 25, 2019
@juanpicado juanpicado force-pushed the fix-118 branch 2 times, most recently from 644f9a3 to 8b671f0 Compare August 25, 2019 13:00
@juanpicado juanpicado added this to the candidate for next release milestone Aug 25, 2019
@juanpicado juanpicado added the bug label Aug 25, 2019
@codecov-io
Copy link

Codecov Report

Merging #121 into master will decrease coverage by 0.09%.
The diff coverage is 50%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #121     +/-   ##
========================================
- Coverage      85%   84.9%   -0.1%     
========================================
  Files          92      92             
  Lines         887     888      +1     
  Branches      157     142     -15     
========================================
  Hits          754     754             
- Misses        115     119      +4     
+ Partials       18      15      -3
Impacted Files Coverage Δ
src/utils/calls.ts 75% <0%> (-25%) ⬇️
src/components/Search/Search.tsx 100% <100%> (ø) ⬆️
src/components/Header/Header.tsx 89.47% <0%> (ø) ⬆️
src/components/Install/Install.tsx 83.33% <0%> (ø) ⬆️
...mponents/RegistryInfoDialog/RegistryInfoDialog.tsx 66.66% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 97e8448...58da99c. Read the comment docs.

@juanpicado juanpicado merged commit ac58730 into master Aug 25, 2019
@juanpicado juanpicado deleted the fix-118 branch August 25, 2019 15:39
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.

WebUI - JWT token not sent search requests
2 participants