Do you think Rahim Khan was right by telling Amir that Hassan was his brother?

I think definitely he was. Secrets had been buried and let fester for much too long. Amir needed to define himself and without this knowledge he could never be whole.