text:
"about me" and "library" buttons are removed

Yeah, its good, however the "/* Hide the nitro ad in the profil tab*/ " removes your ability to change your about me.
( https://imgur.com/a/r7pPOcr )
EDIT: "/*Hide the nitro tab in the homepage */" removes the Library button, that some accounts have
( https://imgur.com/a/5JIvOaH )
copypasted from reddit user TobisDK

Fixed, thank you to you and  TobisDK

authors: CroissantDuNord GOTHMONEY

repo: CroissantDuNord/discord-adblock


text:
Add Bitwise Shift Left Token at translator frontend

Implement the BITWISE_SHIFT_LEFT token to recognize the << operator for performing left bitwise shifts.

@Raghav-2903 will be working on this issue

source: gharchive/issue

repo: CrossGL/crosstl

license_type: permissive

license_source: github-api


text:
Duplicate service value should actually be dataSource

Line https://github.com/CrunchyData/postgres-operator-examples/blob/24f5849ebb8931a6ccabb6341a4a7d81355a911f/helm/postgres/templates/postgres.yaml#L148 should actually be:
dataSource:

I updated it in my downloaded template file and it worked when changing it to dataSource. If you look at the template file it is pretty obvious that this is a typo.

Thanks, this was fixed in https://github.com/CrunchyData/postgres-operator-examples/pull/95

authors: benjaminjb johnf-staybull

repo: CrunchyData/postgres-operator-examples


text:
Show MineCraft Active Online Players

Feature Request
Feature Information:
I'm still trying out AMP and have been really impressed with it and Mike and the community. I run a small BungeeCord Minecraft network for my son and his friends.
I would really like a quick and easy way to at least see the players the players on the network at a glance. This is currently not possible via AMP. It would be greatly appreciated if this feature was implemented. It would be even better if you could also do certain actions to the players.
I don't actually play Minecraft that much, I only manage the server for my son and his friends. So being able to see who's on is important to me. I'm sure others would appreciate this feature too.
I confirm:

[x ] that I have searched for an existing feature request matching the description.

Isn’t there a player list available on the right hand side of the server when you open its console? And you can send commands in the console as well

Yes it does show on the right most of the time.  However when you run a Bungeecord network, it would be really nice to see the players on your network and which server they are on all in one place.  I think it would be fairly easy to implement this feature.

repo: CubeCoders/AMP

license: MIT


text:
🛑 lemmy.best is down

In 090bd74, lemmy.best (https://lemmy.best) was down:

HTTP code: 0
Response time: 0 ms

Resolved: lemmy.best is back up in cd1e3c8 after 5 minutes.


text:
CreateTransferAsync is not saving the Reason field

Hi
On demo and production environments, when calling CreateTransferAsync with the Reason field, the Reason is not saved so when reading back the data with GetTransferAsync or when getting the transfer on the website the field is empty.
Thanks

Thanks for reporting myaron-cs. This is fixed in v2.4.3

authors: EAddario myaron-cs

repo: CurrencyCloud/currencycloud-net

license: mit

license_source: bigquery


text:
压缩无效，返回原图片的相对路径

android 11, 没有使用setTargetDir()方法，压缩后返回原图片的相对路径
代码：
Luban.with(context)
.load(uri)
.ignoreBy(100)
//.setTargetDir(dir.getAbsolutePath())
.filter(path -> {
return !(TextUtils.isEmpty(path));
})
.setCompressListener(onCompressCallback)
.launch();
压缩后的图片：/camera_photos/Pictures/JPEG_20211011_140125.jpg

@Luban Hey guys, that would be nice if you have time to look into this problem, thanks in advance! 🙏🏾
Android 12 has the same problem, as the following code shows：
Luban.with(mContext)
    .load(file)
    .ignoreBy(100)
    .setTargetDir(getPath())
    .setCompressListener(object : OnCompressListener {
        override fun onStart() {}
        override fun onSuccess(compressedFile: File) {
            refreshDataSource(compressedFile.path)
        }

        override fun onError(e: Throwable) {
            e.printStackTrace()
        }
    })
    .launch()
}

Although I invoked the setTargetDir() method to set the compressed file directory, the original image path returned in the callback method onSuccess().

Before compression：/storage/emulated/0/Android/data/com.xxx.test/files/Pictures/JPEG_20220310_2140888612150486718.jpg
After compression：/storage/emulated/0/Android/data/com.xxx.test/files/Pictures/JPEG_20220310_2140888612150486718.jpg
getPath - /storage/emulated/0/Android/data/com.xxx.test/files/compress/

Thanks again and best regards. Looking forward to the fix! 🤝

@Luban Hey guys, hope everything going well! That would be nice if you have time to look into this problem, thanks in advance! 🙏🏾
I encountered the same problem when I was testing on the Android 12 emulator and set back camera from VirtualScene to Emulated(#430 and #428), as the following code shows：
Luban.with(mContext)
    .load(file)
    .ignoreBy(100)
    .setTargetDir(getPath())
    .setCompressListener(object : OnCompressListener {
        override fun onStart() {}
        override fun onSuccess(compressedFile: File) {
            refreshDataSource(compressedFile.path)
        }

        override fun onError(e: Throwable) {
            e.printStackTrace()
        }
    })
    .launch()
}

Although I invoked the setTargetDir() method to set the compressed file directory, the original image path returned in the callback method onSuccess().

Before compression：/storage/emulated/0/Android/data/com.xxx.test/files/Pictures/JPEG_20220310_2140888612150486718.jpg
After compression：/storage/emulated/0/Android/data/com.xxx.test/files/Pictures/JPEG_20220310_2140888612150486718.jpg
getPath - /storage/emulated/0/Android/data/com.xxx.test/files/compress/

Thank you again and best regards. Looking forward to the fix! 🤝

authors: Muran-Hu oudengding

repo: Curzibn/Luban


text:
🛑 StaticJs一（镜像）track.click-it.ca is down

In bd19b4f, StaticJs一（镜像）track.click-it.ca (https://track.click-it.ca/) was down:

HTTP code: 502
Response time: 175 ms

Resolved: StaticJs一（镜像）track.click-it.ca is back up in f9a372d after 14 minutes.

repo: Cutiei/Uptime


text:
Enable XOnline symbol detection when XONLINLS library is found

This enables detection of XOnline symbols when the XONLINLS library is found. I don't know how exactly it differs from XONLINES, but the current signatures seem to be enough for symbol detection to work. The library version seems to always be 5849, and it can be found in games such as TOCA Race Driver 2, Trivial Pursuit and RalliSport Challenge 2.

Thanks for the finding! Just a small favor from your end, can you force-push your branch in order to trigger the CI builds please?

Done, I'm not sure what caused it to not run them before.
The build failures on Windows seem to be caused by VS 2019 not being there, the current Windows image seems to have VS 2022 installed.

Done, I'm not sure what caused it to not run them before.

What caused them not to run before is CI workflow has job scheduler for specific date to perform a release. When repository has no activity for at least 2 months, GitHub will automatically disable workflow containing job scheduler task. I will correct this problem by split CI workflows to only have one workflow has scheduler and other mainly for CI workflow.

The build failures on Windows seem to be caused by VS 2019 not being there, the current Windows image seems to have VS 2022 installed.

Ah, that's right. I forgot about this. I'll need to remove VS build version target and let it run whichever CMake can find.

source: gharchive/pull-request

authors: RadWolfie thrimbor

repo: Cxbx-Reloaded/XbSymbolDatabase


text:
Added new ".VC.db" Visual Studio C++ database file to gitignore

Visual Studio 2015 Update 2 adds a new file format for their internal intellisense db engine, with extension ".VC.db", as described in https://www.visualstudio.com/en-us/news/vs2015-update2-vs.aspx#Cdoubleplus
Anyone installing a new version of 2015 Update 2 will get this new format by default, and will get a ~5MB file in the VS solution folder which should probably not be part of the repo.

Pretty straightforward


text:
Handle docker volumes as a list as well as a dictionary

Docker-py's API allows volumes to be specified as a list or as a dictionary, but currently Vent only supports them a dictionary.  Support for specifying a volume as a list needs to be added.
https://github.com/CyberReboot/vent/blob/master/vent/api/plugin_helpers.py#L700

Hi,
Do you have an example of usage that would follow that code path ?

@sgaist the docker-py documentation for running a container says it takes a parameter for volumes that be either a dict or a list: https://docker-py.readthedocs.io/en/stable/containers.html
they don't provide an example in their docs, so it might take some digging to determine what the docker-py library is expecting.

The container.py  file seems to offer some more details about list handling.
Here is a copy of the interesting part:
**Using volumes**
    Volume declaration is done in two parts. Provide a list of
    paths to use as mountpoints inside the container with the
    ``volumes`` parameter, and declare mappings from paths on the host
    in the ``host_config`` section.
    .. code-block:: python
        container_id = cli.create_container(
            'busybox', 'ls', volumes=['/mnt/vol1', '/mnt/vol2'],
            host_config=cli.create_host_config(binds={
                '/home/user1/': {
                    'bind': '/mnt/vol2',
                    'mode': 'rw',
                },
                '/var/www': {
                    'bind': '/mnt/vol1',
                    'mode': 'ro',
                }
            })
        )
    You can alternatively specify binds as a list. This code is equivalent
    to the example above:
    .. code-block:: python
        container_id = cli.create_container(
            'busybox', 'ls', volumes=['/mnt/vol1', '/mnt/vol2'],
            host_config=cli.create_host_config(binds=[
                '/home/user1/:/mnt/vol2',
                '/var/www:/mnt/vol1:ro',
            ])
        )

based on that, it seems like the list version would be:
['/home/user1':/mnt/vol2:rw', '/var/www:/mnt/vol1:ro']

but it would take some testing to verify.  based on that the code link at the top of this issue needs to be able to handle parsing such and example for attached volumes.  for testing, it's parsing it from the vent.template of the plugin under the docker section.
it should be sufficient to parse it from the above example into a parameter that docker-py can handle though.

authors: cglewis sgaist

repo: CyberReboot/vent


text:
调用loadMoreComplete()会造成无限循环加载更多

如下源码方法：
public void loadMoreComplete() {
        if (getLoadMoreViewCount() == 0) {
            return;
        }
        mLoading = false;
        mNextLoadEnable = true;
        mLoadMoreView.setLoadMoreStatus(LoadMoreView.STATUS_DEFAULT);
        notifyItemChanged(getLoadMoreViewPosition());
    }

当调用notifyItemChanged(getLoadMoreViewPosition());时又会调用onBindViewHolder()方法
此时关键代码来了，此时又重新调用了 autoLoadMore(position);方法，该方法就会出问题
private void autoLoadMore(int position) {
        if (getLoadMoreViewCount() == 0) {
            return;
        }
        if (position < getItemCount() - mPreLoadNumber) {
            return;
        }
        if (mLoadMoreView.getLoadMoreStatus() != LoadMoreView.STATUS_DEFAULT) {
            return;
        }
        mLoadMoreView.setLoadMoreStatus(LoadMoreView.STATUS_LOADING);
        if (!mLoading) {
            mLoading = true;
            if (getRecyclerView() != null) {
                getRecyclerView().post(new Runnable() {
                    @Override
                    public void run() {
                        mRequestLoadMoreListener.onLoadMoreRequested();
                    }
                });
            } else {
                mRequestLoadMoreListener.onLoadMoreRequested();
            }
        }
    }

由于此时调用了loadMoreComplete()方法，mLoading处于false状态，由此会进入下边的判断，造成无限回掉加载更多。。。。。。。这就尴尬了！不光这个方法，loadMoreEnd()....记不清楚了那几个结束加载的方法也是如此，
我遇到的问题场景：对于RecyclerView的item我没有设置满一屏幕，而我设置了加载更多监听，在里边我就设置了上拉加载添加数据，结果竟然有那么多数据，忘记说了我也设置了不满一屏幕时不加载更多虽然有顺序问题，因为在你设置监听时就会开启加载更多，说的有点乱，看一下代码吧！希望有帮助！
 private void init(View view) {
        mRvContent = view.findViewById(R.id.rv_main_content);
        LinearLayoutManager layoutManager = new LinearLayoutManager(getContext());
        mRvContent.setLayoutManager(layoutManager);
        ArrayList<CardBean.CardListEntity> data = new ArrayList<>();
        mAdapter = new SingleItemAdapter(R.layout.my_item, data);
        mRvContent.setAdapter(mAdapter);
        initData();
        if (mAdapter.isLoadMoreEnable()) {
           mAdapter.setOnLoadMoreListener(new BaseQuickAdapter.RequestLoadMoreListener() {
                @Override
                public void onLoadMoreRequested() {
              mAdapter.addData(mCardBean.cardList);
                    mAdapter.loadMoreEnd();
                }
            }, mRvContent);
        }
    }
private void initData() {
        String json = CommenUtils.getJson(NewsApp.getmContext(), "cardlist.json");
        mCardBean = CommenUtils.jsonToBean(json, CardBean.class);
        mAdapter.replaceData(mCardBean.cardList);
        mAdapter.setPreLoadNumber(8);
        mAdapter.disableLoadMoreIfNotFullPage(mRvContent);
    }

上边的mAdapter.disableLoadMoreIfNotFullPage(mRvContent);在这设置没有一点卵用。希望有更好的写法的小伙伴指出不足！

我也遇到这个问题 ，是在SV中嵌套RV时候出现的，在怎么调试都是同样的问题，mloading一直处于false的状态会自动走autoloadmore(pos)这个方法，无限循环。希望作者能处理下

disableLoadMoreIfNotFullPage是跟setNewData一起使用的，只有在没数据第一次设置的时候是有用的

没有数据的时候需要调用loadMoreEnd，如果调了这个就不会回调下一页，请确认是否是在没有数据的时候调了loadMoreEnd方法

@buleSkyvv SV跟RV有冲突，建议添加header方式解决

同上，只有在SV中使用RV时，会自动无限加载，没有找到合适的解决方案，期望新的更新解决

@RyanChouChina @buleSkyv  这个SV嵌套RV是加载机制上的冲突，目前无法解决，建议添加header方式解决或者使用其他上拉的嵌套库。

loadMoreEnd

loadMoreEnd是不是必须在setOnLoadMoreListener监听里面调用呢，如果我返回数据为空，在showData的时候调用loadMoreEnd还是会一直重复调用。

loadMoreEnd

loadMoreEnd是不是必须在setOnLoadMoreListener监听里面调用呢，如果我返回数据为空，在showData的时候调用loadMoreEnd还是会一直重复调用加载更多的方法。

我也是这样？请问有解决方案吗？确实mloading这个值一直处于fasle状态，导致onbindviewholder一直调用autoloadmore()这个方法

如下源码方法：
public void loadMoreComplete() {
        if (getLoadMoreViewCount() == 0) {
            return;
        }
        mLoading = false;
        mNextLoadEnable = true;
        mLoadMoreView.setLoadMoreStatus(LoadMoreView.STATUS_DEFAULT);
        notifyItemChanged(getLoadMoreViewPosition());
    }

当调用notifyItemChanged(getLoadMoreViewPosition());时又会调用onBindViewHolder()方法
此时关键代码来了，此时又重新调用了 autoLoadMore(position);方法，该方法就会出问题
private void autoLoadMore(int position) {
        if (getLoadMoreViewCount() == 0) {
            return;
        }
        if (position < getItemCount() - mPreLoadNumber) {
            return;
        }
        if (mLoadMoreView.getLoadMoreStatus() != LoadMoreView.STATUS_DEFAULT) {
            return;
        }
        mLoadMoreView.setLoadMoreStatus(LoadMoreView.STATUS_LOADING);
        if (!mLoading) {
            mLoading = true;
            if (getRecyclerView() != null) {
                getRecyclerView().post(new Runnable() {
                    @Override
                    public void run() {
                        mRequestLoadMoreListener.onLoadMoreRequested();
                    }
                });
            } else {
                mRequestLoadMoreListener.onLoadMoreRequested();
            }
        }
    }

由于此时调用了loadMoreComplete()方法，mLoading处于false状态，由此会进入下边的判断，造成无限回掉加载更多。。。。。。。这就尴尬了！不光这个方法，loadMoreEnd()....记不清楚了那几个结束加载的方法也是如此，
我遇到的问题场景：对于RecyclerView的item我没有设置满一屏幕，而我设置了加载更多监听，在里边我就设置了上拉加载添加数据，结果竟然有那么多数据，忘记说了我也设置了不满一屏幕时不加载更多虽然有顺序问题，因为在你设置监听时就会开启加载更多，说的有点乱，看一下代码吧！希望有帮助！
 private void init(View view) {
        mRvContent = view.findViewById(R.id.rv_main_content);
        LinearLayoutManager layoutManager = new LinearLayoutManager(getContext());
        mRvContent.setLayoutManager(layoutManager);
        ArrayList<CardBean.CardListEntity> data = new ArrayList<>();
        mAdapter = new SingleItemAdapter(R.layout.my_item, data);
        mRvContent.setAdapter(mAdapter);
        initData();
        if (mAdapter.isLoadMoreEnable()) {
           mAdapter.setOnLoadMoreListener(new BaseQuickAdapter.RequestLoadMoreListener() {
                @Override
                public void onLoadMoreRequested() {
              mAdapter.addData(mCardBean.cardList);
                    mAdapter.loadMoreEnd();
                }
            }, mRvContent);
        }
    }
private void initData() {
        String json = CommenUtils.getJson(NewsApp.getmContext(), "cardlist.json");
        mCardBean = CommenUtils.jsonToBean(json, CardBean.class);
        mAdapter.replaceData(mCardBean.cardList);
        mAdapter.setPreLoadNumber(8);
        mAdapter.disableLoadMoreIfNotFullPage(mRvContent);
    }

上边的mAdapter.disableLoadMoreIfNotFullPage(mRvContent);在这设置没有一点卵用。希望有更好的写法的小伙伴指出不足！

老哥解决这个问题了吗？

@longertime 我现在是这样解决的 在加载更多监听这里判断了下页码

@longertime 我现在是这样解决的 在加载更多监听这里判断了下页码

嗯，可以这么判断，但是我这里需求有点特殊，总页数是变化的，当前页数也是变化的。。。。不过我感觉你的方案是可以的

authors: 1109993488 CymChad RookieJay RyanChouChina buleSkyvv goocher longertime

repo: CymChad/BaseRecyclerViewAdapterHelper


text:
麻烦作者更新下4.x的拖拽功能的文档啊，或者给个demo也行

麻烦作者更新下BRVAH 4.x的拖拽功能（drag and swpie）的文档啊，或者给个demo也行，没文档难受啊

https://github.com/CymChad/BaseRecyclerViewAdapterHelper/wiki/拖拽功能（DragAndSwipeDataCallback）


text:
分组的伸缩栏

分组的伸缩栏中的效果图片没有
url地址：https://github.com/CymChad/BaseRecyclerViewAdapterHelper/wiki/分组的伸缩栏

多谢提醒，现在好了。


text:
Allow phylogeny download as .nex files

Phylogeny download currently produces a .svg file. More useful to allow .nex as well

I've done this, but am pretty sure the code will need to be changed for production @Bibiko @SimonGreenhill as I just set MEDIA_ROOT to /language_trees/. I haven't pushed the changes to Github yet.

By my opinion it would be better to tune the load_all_datasets.sh script to simply copy the folder /language_trees/ to dplace_app/static/trees/ in order to keep the access for static stuff clean.

The other option it to generate a tree file on-the-fly from the currently shown tree and then send the file with the correct Content-Disposition header set (attachment; filename=blah)

I can do what @SimonGreenhill suggested, but the file won't have the list of taxa (just the newick string) that's in the original Glottolog tree files.

Done. Labels in Nexus file are as in the original files; if we save the trees with our society labels in the database (rather than the original trees) then our society labels will be saved in the Nexus file. Closing this as that's a separate issue already.

authors: Bibiko SimonGreenhill michaelcgavin stefelisabeth

repo: D-PLACE/dplace


text:
nspace.d: use foreach

Trivial refactor.

Auto-merge toggled on

authors: WalterBright yebblies

repo: D-Programming-Language/dmd


text:
dub describe includes files from every configuration

If a particular configuration requires dependencies in dub.json, they show up in dub describe even if called with dub --configuration=... describe

Closed since it's a duplicate of this enhancement request

repo: D-Programming-Language/dub


text:
🛑 Vagdepo is down

In 28e95c6, Vagdepo (https://vagdepo.kz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Vagdepo is back up in e9bb67a after 6 minutes.


text:
AttributeError: 'MetricBundle' object has no attribute 'get_classifier_for_attack'

fibber version: fibber 0.2.1 (google colab)
Python version: 3.7
Operating System: windows

Description
AttributeError: 'MetricBundle' object has no attribute 'get_classifier_for_attack'
What I Did
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-8-b1128aa16d94> in <module>()
      5
      6 print("Original sentence:", input_sentence)
----> 7 pred = fibber.get_metric_bundle().get_classifier_for_attack().measure_example(None, input_sentence)
      8 print("Original sentence prediction:", pred)
      9

AttributeError: 'MetricBundle' object has no attribute 'get_classifier_for_attack'

This bug is caused by recent update to the code. The notebook is now fixed.
If you run into any questions, please let me know.

This bug is caused by recent update to the code. The notebook is now fixed.
If you run into any questions, please let me know.

Bug fixed.

repo: DAI-Lab/fibber


text:
Feature request: Example

Hi, thanks for open-sourcing your code! Do you have a video or videogame example? I'm curious what the physics look like and play like.

This asset is based on the physics I use in my project here: Jack of all Cars. Also, there are 4 different examples included that handle very differently.

Oh, great! Thanks!


text:
Bulk data export returns as an array, rather than just a sequence of resources

Bulk data export format should be:
{ resource }
{ resource }
{ resource }
...

But it currently returns as:
[{ resource }{resource }{resource}…]

Please confirm that this issue is resolved.

authors: brynrhodes c-schuler

repo: DBCG/cqf-ruler


text:
Multi-step job support

Classic UI-managed SQL Agent jobs often have multiple steps. Indeed the interface provides a wealth of options for multi-step functionality.
Presently as of v2.0, the tsqlScheduler solution supports only a single jobstep. The typical developer workaround for this seems to be to either

add a series of proc executions in the one available jobstep
wrap multiple procs in a single proc to be called by the jobstep

Error handling and control flow may or may not complicate these implementations. Both of them however impede debugging when things go wrong. Native support for multiple jobsteps lends itself to quicker bug identification than at present.

...use the recent solution from Microsoft

the wat now? O_O
plz link me

I think you originally shared the link with me! Their solution to agent jobs in an AG by syncing the jobs between nodes based on the msdb schéma copies to an AG database (ish).

authors: petervandivier taddison

repo: DBTrenches/tsqlscheduler


text:
error 2149122451   creating update

Hello.
I try to create a update of different msi files ,  and receive in all times the error

thanks.

Hi, can you go to %LocalAppData%\Temp to look at WPP-GUI.log to see if any error message is reported?

WPP-f4e235ad-23f7-4d5c-b776-1a5f55303b3e.log
hello,  tanks for you reply ,   I attached  the file log.

WPP-f4e235ad-23f7-4d5c-b776-1a5f55303b3e.log
this after error

Hi, error 2149122451 map to «Forbidden (403)». Does your WSUS server need to go through a proxy to reach Internet?
Ensure that WPP can access Internet.

Hi, error 2149122451 map to «Forbidden (403)». Does your WSUS server need to go through a proxy to reach Internet?
Ensure that WPP and the WSUS service can access Internet.

Hello,
the serve wsus use a fortigate  to acces to internet.
but  the error is when I try to create a Update from msi file.   not need iinternet

Internet access is required for timestamping the CAB file during the signing.

Yes,  this have internet access,  through a firewall  fortinet

As per the error message, WSUS to be able to reach the Internet. Maybe the firewall blocked it or the WSUS service try to bypass the firewall. Could be an authentication issue as well.

hello @DCourtel ,  thanks for you help
the WSUS work fine ,  and synchronize  without problem  to windows update.  and de WPP is running in the same server.
what connections need open in the firewall to timestamp the CAB  ?

The WSUS server need to reach: http://timestamp.digicert.com
Remember that WSUS runs with the System account. So, if your firewall needs authentication, you need to add the computer account in the access right.
You can also disable timestamping to test, by creating a registry key:

Hello
Work fine !   thanks very much
Regards

authors: ArielBaravalle DCourtel


text:
AB#102119 record decision - change on submit redirect page

Addressing issue raised by @Lisa-V-Sau -
On the summary page, if the answer for 'were any conditions set' is 'No', and is then changed to 'Yes', the follow up question is not presented, hence it will always return empty on the summary page, and clicking 'record a decision' will return a bad request.
As as quick temporary solution; we have agreed that when changing a conditional question, the user will then be required to resubmit all answers from that point onwards this will also include re-entering 'date of decision'.
We understand that it would be optimal to avoid to have the user re-visit questions that have already been answered which can be jarring, so another ticket will be added to the backlog tackle this.

Thanks @jordan-beard that's correct and I've seen the walkthrough already. But I'm not sure if I'm the right person to "approve" the PR as I can't do a code review for you here.

Thanks @jordan-beard that's correct and I've seen the walkthrough already. But I'm not sure if I'm the right person to "approve" the PR as I can't do a code review for you here.

No worries @mingoland. Its just for your visibility.

authors: jordan-beard mingoland


text:
Email candidates with unsuccessful applications before apply again

Context
We want to send a call to action email to all candidates whose applications ended unsuccessfully before we released apply again.
Changes proposed in this pull request

[x] CandidateMailer#apply_again_call_to_action email
[x] Mailer preview for the above
[x] SendApplyAgainCallToAction service class to email all unsuccessful candidates

Guidance to review

I didn't want to over-engineer this as it's a one-off mailshot but do we need a way to ensure that we don't send these emails twice to the same person? e.g. a field in the database.
Do we need a rake task to trigger the service class?
Do we need to parameterise the service class (and rake task) with a datetime for the time that we released the apply again feature or is it ok to assume this will be run soon enough after that happens?

Link to Trello card
https://trello.com/c/yyvDNpCO/1607-dev-build-email-for-candidates-who-were-unsuccessful-when-we-didnt-have-apply-again
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

I didn't want to over-engineer this as it's a one-off mailshot but do we need a way to ensure that we don't send these emails twice to the same person? e.g. a field in the database.

Good question - not sure, but we're not doing any handling for this case in other emails, one-shot or otherwise.

Do we need a rake task to trigger the service class?

I think we're fine to just rails runner the service class on the console.

Do we need to parameterise the service class (and rake task) with a datetime for the time that we released the apply again feature or is it ok to assume this will be run soon enough after that happens?

The latter, IMHO.

I didn't want to over-engineer this as it's a one-off mailshot but do we need a way to ensure that we don't send these emails twice to the same person? e.g. a field in the database.

Good question - not sure, but we're not doing any handling for this case in other emails, one-shot or otherwise.

Do we need a rake task to trigger the service class?

I think we're fine to just rails runner the service class on the console.

Do we need to parameterise the service class (and rake task) with a datetime for the time that we released the apply again feature or is it ok to assume this will be run soon enough after that happens?

The latter, IMHO.

I've added a condition to the query to exclude applications that have already started an apply again. I also added another condition to avoid the duplicate emails - I had forgotten that we log emails so it's fairly easy to do that.

authors: malcolmbaig stevehook


text:
Introduce an 'awaiting references' status after form submission

Context
We need a separate application choice state for when the candidate has submitted the form but the references haven't come back yet. Prior to this PR, the application choice went straight to the 'application complete' state after submission.
Changes proposed in this pull request

Introduce a Reference model
Introduce a new awaiting references state for application choices, that occurs between unsubmitted and application complete:

https://raw.githubusercontent.com/DFE-Digital/apply-for-postgraduate-teacher-training/awaiting_references/docs/states.png

Introduce a Cucumber spec for references coming in
Introduce a ReceiveReference service for assigning incoming feedback to references

Guidance to review
Out of scope: enforcing that submitted applications have 2 references (this will be added in a subsequent PR)
Link to Trello card
https://trello.com/c/bk5Sz72g/1175-simplify-application-statuses-remove-distinction-between-conditional-unconditional-offers
Env vars

no changes

This is fairly far along but I still have a couple of spec failures to follow up FYI @stevehook

@stevehook I've updated the PR based on your feedback, I'd like to leave the "two references present when submitting the application" for a later PR if that's OK, as this one's getting big enough as is.


text:
Update phase banner for different environments

Context
This adds a different phase banner and header colour for each of the 5 environments. It allows us to differentiate between the different versions, and reduces the risk of doing the wrong thing on production.
Changes proposed in this pull request

Guidance to review
Do the colours work? Does the text make sense?
Link to Trello card
https://trello.com/c/0zmKTRYg
Env vars

[x] If this PR introduces new environment variables, they have been added to all necessary parts of the Azure config based on the documentation in the README

We've done this on Publish too, we should try to keep them consistent:
https://github.com/DFE-Digital/manage-courses-frontend/pull/380

(Note that staging is now red instead of brown for reasons of aesthetics, haven't updated the screenshot).

authors: fofr tijmenb


text:
Squish extra whitespace in custom date formats

Update these formats so that single digit hours and days aren't padded
with an extra space.
Context

Changes proposed in this pull request
time = Time.zone.local(2020, 6, 6, 6, 30)

Before
time.to_s(:govuk_date_and_time)
=> " 6 June 2020 at  6:30 am"

After
time.to_s(:govuk_date_and_time)
=> "6 June 2020 at 6:30am"

Guidance to review
Any reason not to?

Link to Trello card

na
Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

@malcolmbaig as we are already using the the no padding flag (-) to achieve the govuk_date format maybe we can update the existing directives so they don't add the extra padding instead.
eg. change '%e %B %Y at %l:%M%P' to %-e %B %Y at %-l:%M%P

@malcolmbaig as we are already using the the no padding flag (-) to achieve the govuk_date format maybe we can update the existing directives so they don't add the extra padding instead.
eg. change '%e %B %Y at %l:%M%P' to %-e %B %Y at %-l:%M%P

@despo Makes sense, but seems simpler to handle whitespace once in each of those lambdas, rather than add multiple flags.

@despo Makes sense, but seems simpler to handle whitespace once in each of those lambdas, rather than add multiple flags.

authors: despo malcolmbaig

repo: DFE-Digital/apply-for-teacher-training


text:
WIP provider/application visibility

Context

Changes proposed in this pull request

Guidance to review

Link to Trello card

Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

 Changing course in support allows both old and new provider to see candidate info


text:
Update privacy notice to include IP address

We are now sending the client IP address to Application Insights so we
state that in the notice.
I've tried to copy the GOV.UK use of IP address in their privacy notice,
I am no expert in this so we may want other eyes on the content?

 Update privacy notice

Thanks for all the comments. We are going to talk to the right people at DfE and get an answer. I'll close this now as it's such a small change, we can open or redo once we have an answer rahter than have this PR going stale.


text:
Removed application insights

Removed application insights .
Terraform Plan
azurerm_application_insights.insights will be destroyed
(because azurerm_application_insights.insights is not in configuration)

resource "azurerm_application_insights" "insights" {

app_id                                = "e435c26e-3543-4c81-8611-1f62402736a2" -> null
application_type                      = "web" -> null
connection_string                     = (sensitive value) -> null
daily_data_cap_in_gb                  = 0.033 -> null
daily_data_cap_notifications_disabled = false -> null
disable_ip_masking                    = false -> null
force_customer_storage_for_profiler   = false -> null
id                                    = "/subscriptions/9816b7b0-58ca-4972-b25b-c7dbd3ee1c6d/resourceGroups/s165d01-rsm-dv-rg/providers/Microsoft.Insights/components/s165d01-rsm-dev-appi" -> null
instrumentation_key                   = (sensitive value) -> null
internet_ingestion_enabled            = true -> null
internet_query_enabled                = true -> null
local_authentication_disabled         = false -> null
location                              = "westeurope" -> null
name                                  = "s165d01-rsm-dev-appi" -> null
resource_group_name                   = "s165d01-rsm-dv-rg" -> null
retention_in_days                     = 30 -> null
sampling_percentage                   = 100 -> null
tags                                  = {

"Environment"      = "dev"
"Parent Business"  = "Teacher Misconduct Unit"
"Portfolio"        = "Early Years and Schools Group"
"Product"          = "Refer Serious Misconduct"
"Service"          = "Teacher Training and Qualifications"
"Service Line"     = "Teaching Workforce"
"Service Offering" = "Refer Serious Misconduct"
} -> null
}

 Remove Application Insights from Ruby apps

repo: DFE-Digital/refer-serious-misconduct


text:
Make clear what we mean by schools

Providers were unclear what we meant by schools, thinking we were talking about placements. This makes the heading here the same as the task list section.

 Make 'Schools' wording on 'Check what data you need' guidance page for SCITTs clearer.


text:
Bardic Inspiration effects not transferring

Bardic Inspiration will work when using CE Panel to transfer effect to actor.
It will not work when using Bardic Inspiration feature from actor menu to transfer onto another actor.
The menu pops up as normal allowing selection. It then goes through proper workflow to place the effect on the targeted actor (Midi). Unfortunately, none of the effects actually transferred just the icon and ID.
This is an issue with nested effects as they all have the same problem. Others that are not nested work just fine.
Could be an issue with Midi, so I can forward this.
10.288
2.0.3

I think this is likely a midi bug. Recommend forwarding it to that module's repo

Fixed in next release

authors: DFreds FreshnessDND

repo: DFreds/dfreds-convenient-effects


text:
Created optimised solution for removing duplicates from sorted array

#2892
i have created a new file giving an optimised solution for removing duplicates from sorted array .
language used :->
java

i have solve the issue #2892 as my first contribution in hacktoberfest


text:
randomly getting broken image

For example this one
https://picsum.photos/500/320/?image=150
but this is fine
https://picsum.photos/500/320/?image=151

Hi,
that's intended, there is no image with the ID 150.
You can find the list of all images here:
https://picsum.photos/images
https://picsum.photos/v2/list

authors: DMarby eldyvoon

repo: DMarby/picsum-photos


text:
Parser gets stuck in infinite loop if final line doesn't contain \n

Hi there,
Firstly thank you for making this gem, I've been playing around with a recipe tool at home and stumbled across cooklang and your parser.
I noticed that if the recipe was missing a new line character at the end of the final line it would hang.
CooklangRb::Recipe.new("A very short recipe")
=> *hangs*

vs
CooklangRb::Recipe.new("A very short recipe\n")
#<CooklangRb::Recipe:0x0000000111c0cb70 @buffer=#<StringScanner fin>, @metadata={}, @source="A very short recipe\n", @steps=[[{"type"=>"text", "value"=>"A very short recipe"}]]>

I've got a branch that I believe fixes the issue (although no particularly elegantly). Would you consider a PR with this fix?

Hey @mathewdbutton, thanks for the bug report! Def feel free to open a PR to fix this, I'd love to take a look

authors: DRBragg mathewdbutton


text:
🛑 Sister Unhas is down

In 67cd8b9, Sister Unhas (http://sister.unhas.ac.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Sister Unhas is back up in a31f709.


text:
🛑 SSO Unhas is down

In 21a9d87, SSO Unhas (https://sso.unhas.ac.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SSO Unhas is back up in 9cd2471 after 17 minutes.


text:
🛑 Apps is down

In 465124f, Apps (https://apps.unhas.ac.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Apps is back up in 8566305 after 5 minutes.


text:
🛑 SSO Unhas is down

In a188bda, SSO Unhas (https://sso.unhas.ac.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SSO Unhas is back up in 94c1556 after 5 minutes.


text:
🛑 SSO Unhas is down

In 50921cc, SSO Unhas (https://sso.unhas.ac.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SSO Unhas is back up in 72a5169 after 5 minutes.


text:
🛑 Dikmawa is down

In e00356a, Dikmawa (https://dikmawa.unhas.ac.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Dikmawa is back up in b98fff7.


text:
🛑 SSO Unhas is down

In b5af7f2, SSO Unhas (https://sso.unhas.ac.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SSO Unhas is back up in 977c61c after 10 minutes.


text:
Cannot query field "github" on type "Query"

I just follow the setup, didn't change anything else.
error Plugin gatsby-source-graphql returned an error
...
  ServerError: Response not successful: Received status code 401
...
error GraphQL Error Unknown field `github` on type `Query`
file: [path]/gatsby-mail/src/components/footer.js

   1 |
   2 |         query FooterQuery {
>  3 |           github {
     |           ^
   4 |             repository(owner: "dschau", name: "gatsby-mail") {
   5 |               stargazers {
   6 |                 totalCount
   7 |               }
   8 |             }
   9 |           }
  10 |           site {
  11 |             siteMetadata {
  12 |               repository {
  13 |                 url
 ERROR  Failed to compile with 1 errors                                                                                12:14:17 AM

 error  in ./src/components/footer.js

Module Error (from ./node_modules/eslint-loader/index.js):

[path]/gatsby-mail/src/components/footer.js
  58:11  error  Cannot query field "github" on type "Query"  graphql/template-strings

✖ 1 problem (1 error, 0 warnings)

 @ ./src/layouts/index.js 19:0-42 71:42-48
 @ ./node_modules/gatsby-plugin-layout/wrap-page.js
 @ ./node_modules/gatsby-plugin-layout/gatsby-browser.js
 @ ./.cache/api-runner-browser-plugins.js
 @ ./.cache/api-runner-browser.js
 @ ./.cache/app.js
 @ multi (webpack)-hot-middleware/client.js?path=/__webpack_hmr&reload=true&overlay=false ./.cache/app

macOS Mojave

This helped.

authors: dnafication oldmannt

repo: DSchau/gatsby-mail


text:
DS-3740 Endpoint to create new submissions

Second step toward the implementation of the workspaceitems endpoint.
This PR needs to be reviewed after #1887

in the DSpace 7 meeting it was decided to just review the final PR #1889 using integration test and curl looking to the doc.
https://wiki.duraspace.org/display/DSPACE/2017-11-30+DSpace+7+Working+Group+Meeting+notes

repo: DSpace/DSpace


text:
DS-2381 sitemap generator skip unreadable items

see DS 2381
added a new Console appender in log4j.properties such that all log DEBUG calls from org.dspace/app appear on the console as well as  dspace.log
personally I find the standard command line utilities rather quiet - I like tracing to assure me that what is happening is actually what I think should be happening

This likely has conflicts with the Service Architecture work.  It will need rebasing and repair.

This needs rebasing on the latest master, because of merge conflicts

Closing as this was replaced by #1289 (see DS-2381 ticket for more details)

authors: akinom mwoodiupui tdonohue


text:
Add the ability to expand / collapse via keyboard

Description
At the moment, the dynamic scrollable dropdown component is not usable by keyboard alone. There's no way to expand the dropdown without resorting to the mouse. This violates success criterion 2.1.1 of the Web Content Accessibility Guidelines (WCAG) 2.1.
This creates a barrier for any keyboard only users looking to complete any submission form that uses the dropdown component.
Instructions for Reviewers
List of changes in this PR:

Add a keydown event to the template.
Add a corresponding keydown event handler to the component

With these changes, you can now interact with the component by:

Tab to a dropdown in a submission form (e.g. dc.type)
With the dropdown focused, push space or enter to toggle the dropdown. Alternatively, you can use the up or down arrow keys to expand the dropdown.

Ideally, this component should mimic the keyboard interactions of a native select element, whereby subsequent presses of the arrow keys should allow the user to select a value from the dropdown. I wasn't able to get this to work, however this at least enables basic keyboard interaction.
In my opinion, the best solution in the long run would be to use a native select element for this component, rather than relying on the ng-bootstrap library. Using a native element will provide support out of the box for keyboard interaction as well as for screen readers.
Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes ESLint validation using yarn lint
[x] My PR doesn't introduce circular dependencies (verified via yarn check-circ-deps)
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR includes new features or configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

The other thing that I will note is that the ng-bootstrap dropdown demo page works properly via the keyboard. So, something about the way it's being used may actually be breaking the dropdown widget. It's possible that we may not have to add our own event handler at all if we can change the way the widget is invoked.
Or, perhaps the version of the library we are using is old and doesn't yet have the keyboard accessibility features?

authors: doana paulo-graca

repo: DSpace/dspace-angular


text:
add e2e tests

Create integration tests for active pages that do not have tests
Add missing tests.

Check out a preview of your pull request here: https://app-servicecanadalabs-dev-pr-preview-0.azurewebsites.net

repo: DTS-STN/Service-Canada-Labs


text:
[Bug] [chunjun-connector-jdbc-base] clickhouse jdbc connection not supported setAutoCommit(false)

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When I tried to implement the synchronization function of clickhouse2hive, I found the following error message.
Caused by: java.sql.SQLFeatureNotSupportedException: Transactions are not supported 		at ru.yandex.clickhouse.ClickHouseConnectionImpl.setAutoCommit(ClickHouseConnectionImpl.java:219) 		at com.dtstack.flinkx.connector.jdbc.source.JdbcInputFormat.openInternal(JdbcInputFormat.java:114)
I think the problem lies in this piece of code

What you expected to happen
I hope to control clickhouse-source to skip opening transactions by configuring parameters,for example:

Because the default value is "semantic": "at-least-once"`, it does not affect other synchronization functions
How to reproduce
When using clickhouse-source to read data, this type of problem may occur
Anything else
null
Version
master
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I think this can unify the management of whether transactions are enabled or not, without the need to change setAutoCommit for every chunjun-connector-module

repo: DTStack/chunjun


text:
base16 colorscheme

Hi,
I'm using the base16 color schemes in my terminal so I made a custom theme for Rainbowstream.
Screenshot on Imgur
If you like it, maybe it can be included.
Anyways, thank you for making this Twitter client, it is absolutely awesome!

👍

authors: DTVD Jorick

repo: DTVD/rainbowstream


text:
onReconnect() does not trigger

I have setup my bot with the following listener
@Override
public void onReady(ReadyEvent event) {
   LOG.info("Connected to Discord");
}

@Override
public void onDisconnect(DisconnectEvent event) {
   CloseCode code = event.getCloseCode();
   String reason = (code == null) ? null : code.getMeaning();
   LOG.info("Disconnected from Discord, reason: {}", reason);
}

@Override
public void onReconnect(ReconnectedEvent event) {
   LOG.info("Successfully reconnected to Discord");
}

While onReady and onDisconnect are properly triggered onReconnect is never called despite the bot managing to reconnect to Discord immediately after a disconnect event has been thrown. The JDA instance is set to automatically reconnect.

onResume is called if a full reconnect is not needed. Resume and Reconnect both imply that the connection has been restored. Resume means it was done within a small enough time frame that the internal cache did not need to be invalidated, and it is a cheap reconnection. Reconnect means that discord requested cache invalidation due to being disconnected for too long. In this case, all JDA entities are destroyed and rebuilt upon reconnecting to discord.
Thus, Resume vs Reconnect.


text:
No such file or directory

@motey
There seems to be an error: [Errno 2] No such file or directory
File "/usr/local/lib/python3.11/site-packages/metaKEGG/modules/pipeline_async.py", line 118, in single_input_genes
entry_dir = os.getcwd()
^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory
I ran one pipeline today, and after that I started getting the error.
I made a fresh conda env with python 3.11.0 and had to addiionally install fsspec
pip install fsspec==2024.10.0
Could you patch that on your end and let me know?
I am preparing the metaKEGG patch version in the meantime

This is an error in MetaKegg. I dont know what https://github.com/dife-bioinformatics/metaKEGG/blob/master/src/metaKEGG/modules/pipeline_async.py#L118 achieves and why it fails...

@motey the patch for metaKEGG under version 1.0.1
https://pypi.org/project/metaKEGG/
https://github.com/dife-bioinformatics/metaKEGG/releases/tag/v1.0.1
Can we try to rebuild the wrapper with this?

@motey Reopened because it persits.
The refactoring of metaKEGG core is imminent.

deployed another attempt of a hotfix 🤞

looks like the new hotfix is working.. will close for now

authors: mlazaratos motey


text:
Please mark this repo as deprecated.

There hasn't been a commit here in a couple years. It seems like the spiritual successor is https://github.com/RustCrypto.

You could send a PR that changes the main README.md file.

The last time I sent a PR changing the README was one year ago tomorrow :-D
Which is fine, projects come and go, no worries. But this project has such a position in Google search results, that it might be nice to mark its status more clearly, one way or another.

Also crypto-hash.
And yes, please, merge that PR, so I immediately know that I need to look elsewhere.
Thanks

Note: Unless we hear otherwise, we are going to officially start tracking this crate as unmaintained in the RustSec Advisory Database:
https://github.com/RustSec/advisory-db/pull/181

repo: DaGenix/rust-crypto


text:
Just a shocking display of snark here John

Look at this double space here in stead of the no space as reviewed on stream.
cc @DaftLimmy

I struggled to see the change at first.
A simple change. Nice.

That space is not even perceptible to the eye.

Can someone merge this PR or does it have to be done by Limmy. It's getting a bit messy with the amount of PRs that are currently open.

authors: Rcjuk adeydev cohan vandelay87

repo: DaftLimmy/TwitchPopups


text:
Using VueQueryPlugin doesn't work for me - bring back useQueryProvider?

I use vue-query in several packaged components that I have. Because they're only components, there is no app instance that I can use the VueQueryPlugin with.  Instead, I've been using useQueryProvider in the top-level component of each package. The result is that there are multiple query providers in the consuming app, but it's been working great.
Now I'm trying to migrate to v2 but I see that useQueryProvider has been removed in favor of only supporting the VueQueryPlugin. This essentially disables my ability to use vue-query at all. Are there any workarounds that you can suggest, or maybe you can bring useQueryProvider back?

Ideally you should not create multiple QueryClient's. Idea is to have one QueryClient in the app to leverage shared cache.
Your components could use useQuery, but do not use provider nor plugin.
The application in which you will be using those components, could have VueQueryPlugin instantiated, which will provide the QueryClient instance trough the component tree.
Other method would be to create QueryClient yourselfin once place and then provide this instance to useQuery composables as an option.

repo: DamianOsipiuk/vue-query


text:
TRPC integration?

Hey man, thanks for all your hard work on vue-query. Awesome to have this tool in Vue now.
Quick question for you: have you ever thought about integrating this with TRPC? TRPC has an integration for react @trpc/react-query would be amazing to have something similar to this for Vue.
Just wondering if you have seen/thought about this and if you have any pointers on how to get this started?
Thanks for your time!

Hi, unfortunately i do not have enough time currently to create and maintain such package.
Maybe when trpc 11 comes up with cleaned up codebase after compatibility layer removal.
If demand for it would be high, maybe someone from the community would step up and create it.

I'm very much interested in tRPC, and believe the benefits it brings to typesafety (alongside an ORM like prisma) are huge.
I'd be interested to work on an adapter for vue-query.

authors: DamianOsipiuk kylegl michealroberts


text:
Getting Started

Love what you have going here!    I was contemplating writing something along these lines since I am trying to use SK with an external API that conditionally uses OAuth2 (based on an initial request-response).    Because of the SSR nature of SK and using hooks.js doesn't seem like the right way to do things for external APIs.
I realize you are still getting this up and running, but do you have a sample svelte.config.js for this?
I am getting issues with @sveltejs/adapter-static and vite complaining that it cannot resolve the package.
Here is what I have:
/** @type {import('@sveltejs/kit').Config} */

import adapter from "@sveltejs/adapter-static";
import * as carbon from "carbon-preprocess-svelte";

const config = {
	preprocess: carbon.presetCarbon(),
	kit: {
		target: '#svelte',
		adapter: adapter(),
		vite: {
			optimizeDeps: { include: ["clipboard-copy"] },
			plugins: [process.env.NODE_ENV === "production" && carbon.optimizeCss()],
			build: {
				rollupOptions: {
					external: [
						"SvelteKitAuth"
					],
				},
			},
		},
	},
};

export default config;

Hi there! Thanks for creating this issue! I'm glad to see more people have been wanting such a library and that I could start working on something like this for SvelteKit!
To clarify: Do you dislike the usage of hooks.{js,ts} in a general implementation or the way SvelteKitAuth does it? If you're referring to the way I designed SvelteKitAuth, I'm open to alternate suggestions that might be more semantic and inline with the way things are otherwise handled in the Svelte ecosystem and would love to see contributions!
Configuring the Library
As for the svelte.config.js, no adjustments need to be made there, as it's simply a dependency that uses API routes as well as the hooks.{js,ts} files to provide endpoints and augment the session which provides all the client-side information to Svelte components.
I just started a enhancements/example-app branch where I am working on getting this setup as a proper dependency for a testing application and you can check out the branch to see how it's configured.
Initially, I was getting errors as well. Since the dependency is built on top of Typescript I had to make some adjustments to the build configuration. Those should be resolved now, though and you can run the example app to see how it's setup!
At the moment, only the getSession and handle methods are configured in the example app. Alongside the main lib/appAuth.ts configuration file where the instance of SvelteKitAuth is created. But you can see how to setup the API routes in the sister repository Cheeri-No. The gist of the setup is a src/routes/api/auth/[...auth].ts file that exports the get and post handlers provided by SvelteKitAuth:
import { appAuth } from "$lib/appAuth";

export const { get, post } = appAuth;

Disclaimer
SvelteKitAuth is still very much in beta. The JWTs are generated with a calculated signature key that can easily be reverse-engineered should you use this and publish the application online. Until configuration options are provided to use a secret key I cannot recommend integrating this in any production applications but as I am working on this for my own use as well those features are already a WIP.

Awesome!   Thanks for adding an example.    I will check it out.
To answer your question.   I like hooks.js in SvelteKit, but it really is really centered around hooking to the current application you are building, not really for when you are creating a wrapper for an external api.
It feels awkward to have to perform your external authentication outside of your library.   In other words, you can't just do something like this:
export default class MyExternalAPI {

    #baseurl;
    #data;
    #sessionID;

    async Init (url) {
        this.#baseurl = url + "/api/v1/"

        let resp = await fetch(this.#baseurl + "getData", {
            method: 'GET',
            headers: {
                Accept: 'application/json',
                'Content-Type': 'application/json',
            },
            credentials: 'same-origin',
        })

        if (resp.status != 200) {
            console.log(`${url} => ${resp.status}:${resp.statusText}`)
            throw new Error(`Unable to connect to TM1 at specified URL [${this.#baseurl}]`)
        }

        // Not possible in SvelteKit
        this.#sessionID = resp.cookie('session_id')

        // Get response data
        this.#data = await resp.json()

    }

    // .....
}

Instead, you have to hook the credentialing outside of the library (as you have done here).    To me....there needs to be a way to do this within SK that is more library friendly.    Going outside of it and requiring an application level hook just feels wrong.
....And thus....why you created SvelteKitAuth    😄
Anyway....I will check out the sample app and see if I can get it running.    I will keep you posted.

I see what you mean. I agree that the hooks in general should be oriented around the application's needs, and not be interfered with too much from external APIs.
Personally, I liked the way session is provided by SvelteKit from the $app/stores module, and I'm not yet sure how that can be recreated. So SvelteKitAuth essentially wraps the getSession hook and you're still free to augment the data with the session() callback, which is designed almost identically to its counterpart in NextAuth.js.
Of course, if you don't want SvelteKitAuth interferring with the SvelteKit session, it's possible to just use the /api/auth/session route it generates instead and fetch the session on load. But I'll look into ways to separate the two in the future!
I suppose it would be possible to make it a svelte.config.js plugin, but that comes with the possible downside that it loses some functionality with classes and environment variables, though I'm not sure there, either, what is supported and what isn't. The design I followed leans on NextAuth.js, with the main difference being the object-oriented approach, allowing far more customization than just adding OAuth2 and HTTP POST providers.
Providing this kind of customizable functionality with zero restrictions and full control to expand with your own authentication methods that might be less conventional than OAuth2 will be a strength of SvelteKit, but comes with the inherit drawback that it needs to be part of your application so you can add custom providers, set the callbacks, etc.

Ok so I see where I was originally having dependency problems. Originally, I did an npm i https://github.com/Dan6erbond/SvelteKitAuth.git and tried to use it like a node_module, which didn't work (might be something wrong with package.json?).

Correct. Not only with the package.json, but there was some issues with the tsconfig.json as well, and some structural changes I had to make in the new branch in order to get it working as a Node module. Now the imports svelte-kit-auth, svelte-kit-auth/client and svelte-kit-auth/providers should work as expected.

It is somehow getting optimized out because I can see in interfaces.ts that it is exporting JWT as well as User and Session

That part is extremely odd. From the code provided I can't see you using JWT at all, especially since you're working in Javascript it should only give you type-hinting at best? But, I don't think that's something I'll have to look into much once I merge enhancements/example-app back into the main branch, which deals with using svelte-kit-auth as a proper Node module, as I said.

BTW: In your example app...NPM v7 has gotten rid of link:... in favor of file:.., so in your package.json, you need to change it to:

Thank you for the tip! I'll give that a look.

I am also getting the same dependency error that I was earlier until I embedded the source.

Hmm. Now this is something I'll have to look into. As you can see, in my example app with Yarn this worked flawlessly and the auth provider functions as expected. Do you mind explaining how exactly you installed it? Could it be a missing dist folder from directly cloning from git? Try running npm run build or yarn build before adding the link.

Small update: I just tried using link:../ instead of file:../, which resulted in a ton of errors with Vite. It seems building for this platform is a bit different than your standard Node modules, so I'll be looking into how other SvelteKit modules do this to hopefully find something better, but for now the standalone installation still seems broken!

Yeah, I think you are seeing the same thing now.   Re-cloned your latest updates, then did an npm install and finally a run build  and you can see that the interfaces.js file is blank.    There is something funky in the config that it is getting optimized out.
It works on your side?   What NPM version are you using?

BTW:  I just noticed this PR....Wonder if that would make this process easier for your project?      https://github.com/sveltejs/kit/pull/1465/commits/2974931a609c2dcde2fd7f7d2f5bb9dba3afc1f5

Yeah, I think you are seeing the same thing now. Re-cloned your latest updates, then did an npm install and finally a run build and you can see that the interfaces.js file is blank. There is something funky in the config that it is getting optimized out.

I'll have to see if using a bundler like microbundle mitigates these issues, otherwise, as I mentioned, I'll check what other SvelteKit packages use and try to adapt the package to those conventions.

It works on your side? What NPM version are you using?

Using Yarn and link: it works flawlessly, but file: installs it locally and that's where Vite throws errors as you can see in #10.

BTW: I just noticed this PR....Wonder if that would make this process easier for your project? sveltejs/kit@2974931

I doubt it. This seems more like an interceptor for the fetch() function that SvelteKit provides, whereas the hook that SvelteKitAuth highjacks is simply a RequestHandler and if you don't want to inject the data into the $app/stores session you can forego the step of assigning the hook in hooks.ts and instead fetch it manually (which I plan on adding as a feature down the road with a client package).

Small update before I close this issue: I've been able to create a bundle now using Rollup with the esbuild and multiple-inputs plugins as you can see in #11, which seems to work without issue using file:../ to install the lib in the demo app.
There are some new issues, though. SvelteKit does not provide access to the router and stores outside of the app itself, from what I can tell, I'll see how to get programmatic navigation, and a separate store working for SvelteKitAuth to use as the client. This means refactor most of the /client submodule, which shouldn't be an issue, as it will allow me to make it class-based and more configurable in case the endpoint changes from /api/auth to something else.
I will see about publishing the first NPM release today, and getting some CI and CD setup in the repository.


text:
Added AudioClip Events and Listener

Summary
Over the last few game jams, I've used a one-off audio playing system each time which listened to an AudioClip GameEvent to allow other systems to be able to play audio in a decoupled way. I've done it enough to where I believe this would be a viable, generally useful addition to the core ScriptableArchitecture library. For this I've created the following:

AudioClipGameEvent
AudioClipUnityEvent
AudioClipGameEventListener

I did not add any icons to the scripts listed above, but that would also be a good thing to do before this is merged in.
In addition, I've hidden a warning via #pragma in the console that appears for private serialized fields in SceneVariable that is a false positive.

Hey, I realize you're probably busy; just bumping this as a reminder.

No worries, I'll probably find time this weekend :)

Appreciate it! Thanks Daniel
Sent with GitHawk

Thanks for the PR! :)

authors: DanielEverland jeffcampbellmakesgames


text:
Tailor output to output terminal capabilities (specifically emacs?)

I just ran cargo script from emacs (which shows the command output in an emacs buffer) and got a load of control characters in the output. By contrast, when I start a fresh cargo project and build cargo build through emacs the output is much cleaner.
A quick test shows that cargo script thing.rs 2> test.txt does clean up the output, but I really don't know who's responsible for that.
-*- mode: compilation; default-directory: "~/code/" -*-
Compilation started at Fri Sep 28 09:31:27

cargo script statgen.rs
^[[0m^[[0m^[[1m^[[32m   Compiling^[[0m statgen v0.1.0
[...]

versus...
-*- mode: compilation; default-directory: "~/code/bob/src/" -*-
Compilation started at Fri Sep 28 09:33:01

cargo build
   Compiling bob v0.1.0 (file:///home/dominic/code/bob)
    Finished dev [unoptimized + debuginfo] target(s) in 0.55s

The output from cargo script looks fine when the build succeeds, I just noticed. cargo build is good in all cases. I'm really not sure where the issue is, here - emacs is clearly capable of showing colour and does so for the succeeding cargo script output.
The behaviour doesn't change when I unload the rust plugins (mode line: Rust ElDoc company racer), so I don't think those are at fault.

repo: DanielKeep/cargo-script


text:
XML parser: single-quote delimited attribute values silently lost

Attributes with their values delimited in single-quotes are lost during loading files. They neither appear in the loaded model nor when serialized again.
<DEFINITION-REF DEST="ECUC-MODULE-DEF">/AUTOSAR_A</DEFINITION-REF>

versus
<DEFINITION-REF DEST='ECUC-MODULE-DEF'>/AUTOSAR_A</DEFINITION-REF>

As of xml specification, single quotes are allowed.

Definition: the content of the AttValue (the text between the ' or " delimiters) as the attribute value.

https://www.w3.org/TR/xml/#sec-starttags
https://www.w3.org/TR/xml/#NT-AttValue

Thanks for the bug report!
The fact that attributes in single quotes didn't work is clearly a bug, though I've never seen a file that was written that way.
I also dislike the fact that it failed silently.
I have fixed both in the commit I just pushed.

authors: DanielT FlTr

repo: DanielT/autosar-data


text:
KeyError: "['target'] not found in axis" using FeatureSelector

When using the FeatureSelector, I get the error as the title shows. The error stack is as follows:
~/env/lib/python3.8/site-packages/verstack/tools.py in wrapped(*args, **kwargs)
     17     def wrapped(*args, **kwargs):
     18         start = time.time()
---> 19         result = func(*args, **kwargs)
     20         end = time.time()
     21         elapsed = round(end-start,5)

~/env/lib/python3.8/site-packages/verstack/FeatureSelector.py in fit_transform(self, X, y, **kwargs)
    230         if self.auto:
    231             self.printer.print(f'Comparing LinearRegression and RandomForest for feature selection', order = 2)
--> 232             self._auto_linear_randomforest_selector(X, y, kwargs)
    233         else:
    234             self.printer.print(f'Running feature selection with {self._model}', order = 2)

~/env/lib/python3.8/site-packages/verstack/FeatureSelector.py in _auto_linear_randomforest_selector(self, X, y, kwargs)
    292
    293         self.printer.print(f'Running feature selection with {linear_model}', order = 2)
--> 294         feats_lr_flags = self._prepare_data_apply_selector(X, y, selector_lr, scale_data = True)
    295
    296         self.printer.print(f'Running feature selection with {randomforest_model}', order = 2)

~/env/lib/python3.8/site-packages/verstack/FeatureSelector.py in _prepare_data_apply_selector(self, X, y, selector, scale_data)
    247
    248     def _prepare_data_apply_selector(self, X, y, selector, scale_data = False):
--> 249         X_subset, y_subset = self._subset_data(X, y)
    250         if scale_data:
    251             X_subset = self._scale_data(X_subset)

~/env/lib/python3.8/site-packages/verstack/FeatureSelector.py in _subset_data(self, X, y)
    484             experimental_data = temp.sample(frac=batch)
    485             experimental_data.reset_index(drop=True, inplace=True)
--> 486             X = experimental_data.drop('target', axis=1)
    487             y = experimental_data.target

I am using v3.3.3 and the source code for FeatureSelector remains unchanged when compared to the newer version

Can you provide the error message? Your stack trace is not complete and does not show the actual error message.
To make it complete also please provide the code how you have applied the FeatureSelector.

You are right with your hypothesis. I've pushed the changes to FeatureSelector to be able to accept any iterable for 'y'.
$pip install --upgrade verstack
You should get verstack version 3.7.1 and FeatureSelector version 0.0.5

repo: DanilZherebtsov/verstack


text:
What I miss

Hi this is a very interesting plug-in, with lots of potential. If I may make the following suggestions - I would be very grateful.

The area where one adds variables ... scrolls down as one adds variables and then becomes unseen when there are too many. Could it not rather be at the top and variables are add below it ?
Could the variables list have a scroll bar so one can go down the list and change values with out going into the %NoteVariables% file ? (the %NoteVariables% file is too confusing for a non programmer like me)
This may be asking too much, but this plug in is very useful for different types of notes / documents, so if at the beginning of a note or document one could refer to a specific: %NoteVariables - XXX%, then different files could have %NoteVariables% ... this would save time.

Thank you very much for your attention and time -much appreciated.
Sergio

Points 1 and 2: I removed the right panel on v2.0.0, now the variables can be edited directly on the Variables Note using the Markdown table format, please update to the latest version and check the Usage.
Point 3 is now implemented on v2.0.0 ;)

Thank you !
Can't wait to try it out !!
Best regards,
Sérgio

authors: DanteCoder sergioflorespereira

repo: DanteCoder/JoplinPluginNoteVariables


text:
Version discrepancy installation on Databricks

When installing the latest version of flash-attn from the Git repository (2.2.3) - we have installed this manually on databricks, however when checking the library version this is coming back with 1.0.7
Version in databricks identified through pip freeze - the latest version was imported directly from https://github.com/Dao-AILab/flash-attention/releases/tag/v2.3.3

Sorry I'm not super familiar with how things work on databricks. Can't you just do pip install flash-attn==2.3.3?

So this doesn’t work for our use case, whereby we need to install the libraries directly using the wheels (the env is off network). Do you know why the version of flash-attn would be seen as 1.0.7 even though it’s the latest from the git repository (2.3.3) as this is causing dependency issues

Idk the version on PyPI is also 2.3.3.

@KPMGLH  pip allows multiple version of the same pkg installed. We have seen bad "version" logic from pip choosing the wrong flash-attn module. For example, it chose 2.2.x when 2.3.2 was also installed. There is a problem when both wheel version and manual pip git+https are used to install modules.
My suggestion, always "pip uninstall" the old flash-attn before installing new one.

authors: KPMGLH Qubitium tridao

repo: Dao-AILab/flash-attention


text:
VSL-51: Added new asset flow - BE

Description

This PR is about adding assets over safe settings. These assets can be vaults or collections.
In case of vaults, user needs to provide the name of the contract (FlowToken, FiatToken etc)
In case of collections, user needs to provide both the name and the address of the contract.
These values will be used to trigger the correct transaction and add vault/collection to the treasury.
Added small piece of UI for enabling users to add assets.
Added other tokens for testing on staging - Starly, Blocto and REVV

Demo / Test Result
Safe Settings, below the Owners section, two buttons are added:

Add Vault Modal:

### Add Collection Modal:

For now only add collection can be tested by using ExampleNFT contract on local testing environment, or on staging environment - any NFT contract deployed to testnet.
Adding vaults will be testable once we update flow.json with the names of the contracts and addresses of the accounts they are deployed, similar to how it is done in the next snapshot:

@KsenijaZivkovic , after user added Vault, it needs to be rendered under coin type dropdown: https://github.com/DapperCollectives/VESSEL/blob/5efbfc90581a863ce7eb99989b90d2ec199e414e/packages/client/src/components/SendTokens/components/CoinTypeDropDown.js#L9, can that be covered in this PR or we create another ticket for that work?

@KsenijaZivkovic , after user added Vault, it needs to be rendered under coin type dropdown:
https://github.com/DapperCollectives/VESSEL/blob/5efbfc90581a863ce7eb99989b90d2ec199e414e/packages/client/src/components/SendTokens/components/CoinTypeDropDown.js#L9
, can that be covered in this PR or we create another ticket for that work?

Changed from text input to dropdown!

repo: DapperCollectives/VESSEL


text:
feat: add endpoints to fetch the most recent PvP kills

Another endpoints should be added that returns all recent PvP kills similar to /v-rising-discord-bot/player-activities.

closed in favor of: https://github.com/DarkAtra/v-rising-discord-bot-companion/pull/11

repo: DarkAtra/v-rising-discord-bot-companion


text:
not working on 1.1.2 version of pytho_aternos

i downgrade to 1.1.2 but it still has the problem

Hello. You should update to the latest version (2.1.3). What problem do you have?


text:
Add alternative IMFs

Currently we only have Chabrier, we should at least add Kroupa.

@sazabi4 once mentioned some interest in this.

I think it's both IMF and isochrone. We should add in an empirical isochrone as well as IMF to

make the isochrone best representative to the dwarf galaxies in the RGB region. @kadrlica has sort of started this a while ago. I found a plot I made a long time ago and may help some of the work on this.

Probably this first one should go to a different issue thread.

make the isochrone/IMF go brighter than Mv = -2. I think the TRGB should go up to Mv = -4 but I think the padova isochrone goes to Mv = -3 but somehow the Chabrier IMF doesn't give anything at Mv = -2.5 or brighter. I'm not sure if that's true and I think further investigation is required.

Just in case you are interested, we brought up the 2nd discussion at the LSST Hack. This question was brought up as it affects the understanding of the furthest dwarf galaxy in Local Group that can have resolved stars observed by LSST.

The empirical isochrones are actually a different issue (#21).

repo: DarkEnergySurvey/ugali


text:
Adding host metadata to the inventory payload

What does this PR do?
Adding host metadata to the inventory payload. This will be used in the backend to populate the inventories product.
Describe how to test/QA your changes
Start the Agent and wait for the inventory payload to be sent (this can be configured to a shorter time using  DD_INVENTORIES_MIN_INTERVAL and DD_INVENTORIES_MAX_INTERVAL.
Then check that the payload was sent successfully and that it contains the correct values. datadog-agent troubleshooting metadata_inventory command can be use to show the latest payload sent.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

This PR makes me think of something else actually: when enable_metadata_collection is disabled, I think we want the Agent not to send inventories payloads either (because, similar to the "legacy" metadata payloads, the inventories payloads should only be sent by at most one agent per host). WDYT? If you agree, can you make sure this is ensured today?

This PR makes me think of something else actually: when enable_metadata_collection is disabled, I think we want the Agent not to send inventories payloads either (because, similar to the "legacy" metadata payloads, the inventories payloads should only be sent by at most one agent per host). WDYT? If you agree, can you make sure this is ensured today?

It's a good question: the agent_metadata and check_metadata might be different from agent to agent and we might want them. The host_metadata should only be sent once. I think we need finer control for this. WDYT ?

authors: hush-hush olivielpeau

repo: DataDog/datadog-agent


text:
[Backport 7.53.x] [CWS] only report ebpfless selftest once

What does this PR do?
This PR backports commit 7ec22b4 from #24358 and commit 21afaa7 from #24368 to the 7.53.x branch.

Motivation
The goal of these backports is to fix the reporting of ebpfless selftests that were incorrectly run twice. This would cause the second run to be reported as failing.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

authors: YoannGh paulcacheux


text:
Finish renaming to installer binaries

What does this PR do?

move initial datadog-installer to its own dir: not in /opt/datadog to avoid interacting with other projects, not in /opt/datadog-packages as all files there are managed by the installer
finish renaming all bin paths to installer

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge


text:
[BARX-598] Deploy new 2022 and 2019 windows images

What does this PR do?
This PR creates new ltsc2019 and ltsc2022 images.
Motivation
See associated card + https://github.com/containerd/containerd/issues/6508
Describe how to test/QA your changes
Deployed as part of this pipeline (required a backport + custom image) https://gitlab.ddbuild.io/DataDog/datadog-agent/-/pipelines/49488013. You can check that all the images listed in the release notes are published and that the manifest matches only a single platform. Example:
❯ crane manifest docker.io/datadog/agent-dev:ali-propose-new-win-images-py3-win-ltsc2019 | jq
{
  "schemaVersion": 2,
  "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
  "manifests": [
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 7829,
      "digest": "sha256:b4eeeea461781758903f05316a49886943adfa15001157205a202ee78ef1833d",
      "platform": {
        "architecture": "amd64",
        "os": "windows",
        "os.version": "10.0.17763.6414"
      }
    }
  ]
}
❯ crane manifest docker.io/datadog/agent-dev:ali-propose-new-win-images-py3-win-servercore-ltsc2019 | jq
{
  "schemaVersion": 2,
  "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
  "manifests": [
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 6114,
      "digest": "sha256:ea087800300e2b8ec08af4397480db190f194ffd9ae4d40b2af7b04638992f31",
      "platform": {
        "architecture": "amd64",
        "os": "windows",
        "os.version": "10.0.17763.1697"
      }
    }
  ]
}

This can only be check on new release candidates
Possible Drawbacks / Trade-offs
Additional Notes

/merge


text:
After upgrade from v1.4.0 to v1.5.0 got many vulnerabilities

pkg:golang/github.com/go-ldap/ldap/v3@v3.1.10
1 known vulnerabilities affecting installed version
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ 1 vulnerability found                                                                            ┃
┣━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Description        ┃ 1 non-CVE vulnerability found. To see more details, please create a free    ┃
┃                    ┃ account at https://ossindex.sonatype.org/ and request for this information  ┃
┃                    ┃ using your registered account                                               ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ OSS Index ID       ┃ sonatype-2020-1055                                                          ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Score         ┃ 8.1/10 (High)                                                               ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Vector        ┃ CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:U/C:H/I:H/A:N                                ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Link for more info ┃ https://ossindex.sonatype.org/vulnerability/sonatype-2020-1055              ┃
┗━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
pkg:golang/github.com/gofiber/fiber/v2@v2.11.0
1 known vulnerabilities affecting installed version
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ 1 vulnerability found                                                                            ┃
┣━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Description        ┃ 1 non-CVE vulnerability found. To see more details, please create a free    ┃
┃                    ┃ account at https://ossindex.sonatype.org/ and request for this information  ┃
┃                    ┃ using your registered account                                               ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ OSS Index ID       ┃ sonatype-2021-4750                                                          ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Score         ┃ 6.5/10 (Medium)                                                             ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Vector        ┃ CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:U/C:N/I:N/A:H                                ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Link for more info ┃ https://ossindex.sonatype.org/vulnerability/sonatype-2021-4750              ┃
┗━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
pkg:golang/github.com/hashicorp/vault/api@v1.1.0
1 known vulnerabilities affecting installed version
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ 1 vulnerability found                                                                            ┃
┣━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Description        ┃ 1 non-CVE vulnerability found. To see more details, please create a free    ┃
┃                    ┃ account at https://ossindex.sonatype.org/ and request for this information  ┃
┃                    ┃ using your registered account                                               ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ OSS Index ID       ┃ sonatype-2021-3619                                                          ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Score         ┃ 7.5/10 (High)                                                               ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Vector        ┃ CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H                                ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Link for more info ┃ https://ossindex.sonatype.org/vulnerability/sonatype-2021-3619              ┃
┗━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
pkg:golang/github.com/jinzhu/gorm@v1.9.1
1 known vulnerabilities affecting installed version
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ [CVE-2019-15562] CWE-89: Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection')                                                                                ┃
┣━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Description        ┃ ** DISPUTED ** GORM before 1.9.10 allows SQL injection via incomplete                                                                                ┃
┃                    ┃ parentheses. NOTE: Misusing Gorm by passing untrusted user input where Gorm                                                                                ┃
┃                    ┃ expects trusted SQL fragments is a vulnerability in the application, not in                                                                                ┃
┃                    ┃ Gorm.                                                                                ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ OSS Index ID       ┃ CVE-2019-15562                                                                                ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Score         ┃ 9.8/10 (Critical)                                                                                ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Vector        ┃ CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H                                                                                ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Link for more info ┃ https://ossindex.sonatype.org/vulnerability/CVE-2019-15562?component-type=golang&component-name=github.com%2Fjinzhu%2Fgorm&utm_source=nancy-client&utm_medium=integration&utm_content=1.0.37 ┃
┗━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
pkg:golang/github.com/urfave/negroni@v1.0.0
1 known vulnerabilities affecting installed version
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ 1 vulnerability found                                                                            ┃
┣━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Description        ┃ 1 non-CVE vulnerability found. To see more details, please create a free    ┃
┃                    ┃ account at https://ossindex.sonatype.org/ and request for this information  ┃
┃                    ┃ using your registered account                                               ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ OSS Index ID       ┃ sonatype-2021-1485                                                          ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Score         ┃ 6.1/10 (Medium)                                                             ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ CVSS Vector        ┃ CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:L/I:L/A:N                                ┃
┣━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Link for more info ┃ https://ossindex.sonatype.org/vulnerability/sonatype-2021-1485              ┃
┗━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛

Please update dependencies

From what I can tell (go mod graph), all 5 vulnerabilities come from gopkg.in/DataDog/dd-trace-go. The latest version (v1.39.0 still depends on the same packages). I will reach out to the team that manages that repo and will bump the version once they update.

dd-trace-go depends on github.com/hashicorp/vault/api v1.1.0
https://github.com/DataDog/dd-trace-go/blob/e5fb07470a220a3a31759a986561afb0440a18da/go.mod#L55

Hi @MRabenda, I've bump the dependency of dd-trace-go to the latest version, which addresses the concerns. Thanks for letting us know!

authors: MRabenda kimi-p

repo: DataDog/datadog-lambda-go


text:
Don't unpatch HTTP lib if the Datadog tracer is initialized

What does this PR do?
Prevents the layer from unpatching the http lib when the DD tracer is present. When the DD tracer is initialized it handles patching/unpatching on its own.
Motivation
Unpatching the http lib after function invocations complete was causing the layer to remove the patching that the DD tracer was doing, resulting in correct traces for cold starts but missing HTTP request spans in subsequent executions.
Testing Guidelines
Tested the new version of the layer with our test Lambdas and confirmed that the issue was fixed.

Codecov Report

Merging #52 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master   #52      +/-   ##
=======================================
+ Coverage   92.99%   93%   +<.01%
=======================================
  Files          25    25
  Lines         714   715       +1
  Branches      116   116
=======================================
+ Hits          664   665       +1
  Misses         30    30
  Partials       20    20

Impacted Files
Coverage Δ

src/trace/listener.ts
92.15% <100%> (+0.15%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 291ff1d...577c32e. Read the comment docs.

authors: codecov-io sfirrin

repo: DataDog/datadog-lambda-layer-js


text:
MTG-8 Apply new detekt configuration

What does this PR do?
Use the shared static analysis job with an updated detekt configuration

Codecov Report

Merging #1180 (32a07a0) into develop (84835ea) will increase coverage by 0.02%.
The diff coverage is 87.27%.

@@             Coverage Diff             @@
##           develop    #1180      +/-   ##
===========================================
+ Coverage    82.25%   82.26%   +0.02%
===========================================
  Files          353      353
  Lines        11777    11790      +13
  Branches      2005     2005
===========================================
+ Hits          9686     9699      +13
- Misses        1474     1475       +1
+ Partials       617      616       -1

Impacted Files
Coverage Δ

.../com/datadog/android/ktx/coroutine/CoroutineExt.kt
75.86% <ø> (ø)

.../kotlin/com/datadog/android/ktx/rum/ClosableExt.kt
100.00% <ø> (ø)

...om/datadog/android/ktx/sqlite/SqliteDatabaseExt.kt
92.86% <ø> (ø)

.../kotlin/com/datadog/android/ktx/tracing/SpanExt.kt
86.67% <ø> (-1.57%)
:arrow_down:

...in/com/datadog/android/sqldelight/SqlDelightExt.kt
80.65% <ø> (ø)

...rnal/persistence/file/EncryptedFileReaderWriter.kt
93.75% <ø> (ø)

...ndroid/core/internal/persistence/file/FileMover.kt
62.96% <ø> (ø)

...rnal/persistence/file/batch/BatchFileDataWriter.kt
88.24% <ø> (ø)

...rsistence/file/batch/PlainBatchFileReaderWriter.kt
88.24% <ø> (+1.96%)
:arrow_up:

...al/persistence/file/single/SingleItemDataWriter.kt
95.83% <ø> (ø)

... and 71 more

authors: codecov-commenter xgouchet

repo: DataDog/dd-sdk-android


text:
Fix integration tests pipeline for API 21

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.98%. Comparing base (2529684) to head (c6765b9).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2197      +/-   ##
===========================================
+ Coverage    69.93%   69.98%   +0.05%
===========================================
  Files          726      726
  Lines        26967    26999      +32
  Branches      4526     4526
===========================================
+ Hits         18858    18895      +37
  Misses        6830     6830
+ Partials      1279     1274       -5

see 28 files with indirect coverage changes