Problem/Motivation

A new (last) beta release for Social Auth is to be released in the following days. There have been some new things added to Social Auth such as a base controller for OAuth2: #2982765: Provide base controller class for OAuth2 authentication. Furthermore, there was an API change introduced by #3027007: [META] Allow developers to specify the HTTP method. It would be nice to have a compatible version ready for Social Auth Vk for the release date.

Proposed resolution

Update controller and VkontakteAuthManager

CommentFileSizeAuthor
#3 3027219-3.patch21.22 KBgvso
#2 3027219.patch21.2 KBgvso

Comments

gvso created an issue. See original summary.

gvso’s picture

Issue summary: View changes
StatusFileSize
new21.2 KB
gvso’s picture

StatusFileSize
new21.22 KB
davps’s picture

Assigned: Unassigned » davps
Status: Active » Needs review

@3gvso thanks for attached patch. I'll review and test it.

Status: Needs review » Needs work

The last submitted patch, 3: 3027219-3.patch, failed testing. View results

gvso’s picture

Status: Needs work » Needs review

I created a new release for Social Auth. Any updates on this?

davps’s picture

@gvso sorry, there are no updates by issues. I'll try to check them asap.

  • gvso authored cf1f614 on 8.x-2.x
    Issue #3027219 by gvso, davps: Update module to current Social Auth API
    
ugolek’s picture

Status: Needs review » Reviewed & tested by the community

Hi guys, I have tested this patch, seems all is working well.

gvso’s picture

Changed to fixed because the patch was committed. Also, I would recommend adding a new release

gvso’s picture

Status: Reviewed & tested by the community » Fixed
ugolek’s picture

Status: Fixed » Closed (fixed)
davps’s picture

Assigned: davps » Unassigned