Skip to content

Add fallback source type to handle errors#156

Closed
tbonfort wants to merge 3 commits intoMapServer:mainfrom
tbonfort:fallback-source
Closed

Add fallback source type to handle errors#156
tbonfort wants to merge 3 commits intoMapServer:mainfrom
tbonfort:fallback-source

Conversation

@tbonfort
Copy link
Copy Markdown
Member

@tbonfort tbonfort commented Jan 17, 2017

In the same vein as fallback caches, this PR adds support for fallback sources. If the first configured source fails, mapcache will cycle the remaining configured sources until/if one succeeds. If all fail, the error message of the first failing source will be returned.

example usage/config:

<source name="wms1" type="wms">
 <!-- wms 1 config opts -->
</source>

<source name="wms2" type="wms">
 <!-- wms 2 config opts. Most probably the same as for wms except for the server hostname, unless you know what you are doing -->
</source>

<source name="fallbackwms" type="fallback">
 <source>wms1</source>
 <source>wms2</source>
</source>

<tileset name="foo">
 <source>fallbackwms</source>
 <!-- ... -->
</tileset>

@tbonfort
Copy link
Copy Markdown
Member Author

cc @tevbrasch

Base automatically changed from master to main January 15, 2021 22:57
@jbo-ads
Copy link
Copy Markdown
Member

jbo-ads commented Mar 10, 2021

@tbonfort - Is this pull request still relevant?

@tbonfort
Copy link
Copy Markdown
Member Author

@tevbrasch are you still using this? If so please create a new PR based against the current main branch so it can be included.

@tbonfort tbonfort closed this Mar 11, 2021
@tevbrasch
Copy link
Copy Markdown

Hi @tbonfort , yes we're still using this. But we haven't made a any additions or changes, so not sure why it needs a new PR?

@jmckenna
Copy link
Copy Markdown
Member

Thanks @tevbrasch , this does look like it would be very useful to the community, and it was already re-based to the main branch automatically. We should re-open this ticket and set the target to the upcoming 1.12.0 release. For some reason I am unable to re-open here, can someone else re-open this ticket?

@jmckenna jmckenna added this to the 1.12.0 milestone Mar 15, 2021
@tbonfort
Copy link
Copy Markdown
Member Author

Hi @tbonfort , yes we're still using this. But we haven't made a any additions or changes, so not sure why it needs a new PR?

@tevbrasch for this to be included in an upcoming release, someone has to take the time and effort to fix the merge conflicts. Once that's done the result can be pushed to a new PR and be milestoned.

@jmckenna
Copy link
Copy Markdown
Member

@tevbrasch I don't have a local copy of these changes, as unfortunately the directory with the changes was just deleted. Would you mind filing a new pull request with those changed files? thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants