Page MenuHomePhabricator

Fix HTTP service issue
AbandonedPublic

Authored by WyldBot on Oct 4 2019, 7:05 PM.

Details

Reviewers
Info-Screen
Summary

Ensures API requests end in a trailing slash

Test Plan

Test client

Diff Detail

Repository
rWCL Web Client
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 702
Build 702: arc lint + arc unit

Event Timeline

Hackintosh5 added inline comments.
src/app/_services/http.service.ts
28

Is a format string sensible? I think in this case it's more sensible to just append a '/', although I don't know TypeScript

181–182

This seems very wrong. Shouldn't this be done by creating a dictionary and setting a key if the token is present?