I don't see the ```url``` accessor used in the code. There is only the variable ```@url``` assignment in the constructor which gets a value from a constant. From what I see the ```url``` attr_reader is also not needed as you use it only once inside a private method ```announce_handler``` here https://github.com/yast/yast-registration/pull/2/files#diff-ba56a0088ec252bb5929cac2d8ab9602R48 , where you could use the constant. I'm not sure how much is the current code going to change, it not much, I would get rid of it.