m3dev / octoparts

Octoparts, the backend services aggregator
https://m3dev.github.io/octoparts/
Other
151 stars 18 forks source link

refs !180; Proxy overriding - fixed on review comments (Re: #187) #189

Closed reki2000 closed 8 years ago

reki2000 commented 8 years ago

Thank you for reviewing!

Done:

Progress:

@lloydmeta , As for using repository to keep proxy settings, I suggest that this time how about introduce this feature (with application.conf) as "experimental" state.

Because @xevix is also telling that setting might be more flexible, I agree with that idea, but I need to try this feature on our testing environment to check if it works as expected (including tester operation).

I guess we'll have another finding from that operational test.

EDIT: I thought that if using in "experimental" state, changing client API might be affects too much(newRequest supposed to require proxyId) . Another idea is to use "additionalHeader" argument to specify proxyId by "X-OCTOPARTS-PROXY-ID" header. This doesn't need updating octoparts jar in client application.

xevix commented 8 years ago

Hey.

Just to clarify, what I was referring to with part/service/service group, is to store that information in the database, not in Octoparts' config/settings. As this has nothing to do with Octoparts in general, but rather particular parts in specific, this information should be stored in the database, just as part information is stored.

If I understand you correctly, you want to try this first with config only. In this case, we can just deploy the branch with the changes locally. In this case, perhaps we should continue this discussion internally, rather than on GitHub. What do you think?

mauhiz commented 8 years ago

You have something to hide? huh? :shipit:

lloydmeta commented 8 years ago

@mauhiz :speak_no_evil:

xevix commented 8 years ago

@mauhiz Top-secret level 9000 classified plans. I've already said too much.

lloydmeta commented 8 years ago

Closing this for now to prevent accidental merging