spring-cloud / spring-cloud/spring-cloud-bus

Add Gateway routes refresh support

Open
#57 1 comment 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Java
Stars
445
Forks
246
Avg merge
16h 25m
Merged PRs (30d)
5

Description

We're using a config server and Spring Cloud Bus to distribute a RefreshRemoteApplicationEvent. This works fine so far.
When our Zuul Proxy receives the event with changed routes they don't get loaded.
I've solved this by adding my own RefreshListener which does the following:
zuulHandlerMapping.setDirty(true);

After adding this the refresh works fine and new routes are added.

We're on Cameden SR5.
It's quite a simple setup.
One config server.
One Zuul server.
One RabbitMQ.

To reproduce add a route to the config and invoke the /monitor endpoint on the configserver.
The Zuul gets triggered but the routes are not changed.

If you need any further information feel free to ask anytime.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing RefreshRemoteApplicationEvent handling from the config server's /monitor endpoint through the Zuul route refresh path, focusing on the reported RefreshListener and ZuulHandlerMapping.setDirty(true) behavior. Reproduce with a config server, Zuul server, RabbitMQ, and a newly added route; done means the route changes are loaded after the refresh event.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, rabbitmq, spring-boot
Domain
backend, distributed-systems
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.